[wip] pointer decompression no-op [v8/v8 : main]

0 views
Skip to first unread message

Steinar H Gunderson (Gerrit)

unread,
Dec 19, 2025, 4:29:03 AM (10 days ago) Dec 19
to v8-re...@googlegroups.com, oilpan-r...@chromium.org, cbruni...@chromium.org

Steinar H Gunderson has uploaded the change for review

Commit message

[wip] pointer decompression no-op
Change-Id: I1480f0032e3733972553b1e34eb4a7def5f157ed

Change diff

diff --git a/include/cppgc/internal/member-storage.h b/include/cppgc/internal/member-storage.h
index f862740..7fff2ea 100644
--- a/include/cppgc/internal/member-storage.h
+++ b/include/cppgc/internal/member-storage.h
@@ -154,6 +154,14 @@
// the cage alignment.
CPPGC_DCHECK((!compressed || compressed == kCompressedSentinel) ||
(compressed & (1 << 31)));
+
+ // Tell the compiler that decompressing this compressed pointer
+ // is a no-op. Note that Clang is unable to inline Decompress()
+ // into an assume, so we need to write it out fully.
+ [[assume(((static_cast<uint64_t>(static_cast<int32_t>(compressed))
+ << api_constants::kPointerCompressionShift) &
+ base) == uptr)]];
+
return compressed;
}

Change information

Files:
  • M include/cppgc/internal/member-storage.h
Change size: XS
Delta: 1 file changed, 8 insertions(+), 0 deletions(-)
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: newchange
Gerrit-Project: v8/v8
Gerrit-Branch: main
Gerrit-Change-Id: I1480f0032e3733972553b1e34eb4a7def5f157ed
Gerrit-Change-Number: 7277326
Gerrit-PatchSet: 1
Gerrit-Owner: Steinar H Gunderson <se...@chromium.org>
unsatisfied_requirement
open
diffy

chromeperf@appspot.gserviceaccount.com (Gerrit)

unread,
Dec 19, 2025, 5:27:02 AM (10 days ago) Dec 19
to Steinar H Gunderson, cbruni...@chromium.org, oilpan-r...@chromium.org, v8-re...@googlegroups.com
Attention needed from Steinar H Gunderson

Message from chrom...@appspot.gserviceaccount.com

📍 Job mac-m1_mini_2020-perf/speedometer3 complete.

See results at: https://pinpoint-dot-chromeperf.appspot.com/job/13203474b10000

Open in Gerrit

Related details

Attention is currently required from:
  • Steinar H Gunderson
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: v8/v8
Gerrit-Branch: main
Gerrit-Change-Id: I1480f0032e3733972553b1e34eb4a7def5f157ed
Gerrit-Change-Number: 7277326
Gerrit-PatchSet: 1
Gerrit-Owner: Steinar H Gunderson <se...@chromium.org>
Gerrit-Attention: Steinar H Gunderson <se...@chromium.org>
Gerrit-Comment-Date: Fri, 19 Dec 2025 10:26:59 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages