Skip to content

Commit d2b19ba

Browse files
Dominik InführV8 LUCI CQ
authored andcommitted
[compiler] Emit proper instruction on indirect write barrier
When skipping indirect write barriers, we need to emit kArchStoreIndirectSkippedWriteBarrier which triggers the right verification. Bug: 437096305, 445102886 Change-Id: Ib4f3161a78fea1b7ce605e44a9484acd0fd825ba Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6954306 Reviewed-by: Darius Mercadier <[email protected]> Commit-Queue: Dominik Inführ <[email protected]> Cr-Commit-Position: refs/heads/main@{#102524}
1 parent 36dda0d commit d2b19ba

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/compiler/backend/arm64/instruction-selector-arm64.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1279,8 +1279,8 @@ void InstructionSelector::VisitStore(OpIndex node) {
12791279
write_barrier_kind == kSkippedWriteBarrier);
12801280
// In this case we need to add the IndirectPointerTag as additional input.
12811281
code = write_barrier_kind == kSkippedWriteBarrier
1282-
? kArchStoreIndirectWithWriteBarrier
1283-
: kArchStoreIndirectSkippedWriteBarrier;
1282+
? kArchStoreIndirectSkippedWriteBarrier
1283+
: kArchStoreIndirectWithWriteBarrier;
12841284
code |= RecordWriteModeField::encode(
12851285
RecordWriteMode::kValueIsIndirectPointer);
12861286
IndirectPointerTag tag = store_view.indirect_pointer_tag();

src/compiler/backend/x64/instruction-selector-x64.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1478,8 +1478,8 @@ void VisitStoreCommon(InstructionSelector* selector,
14781478
write_barrier_kind == kSkippedWriteBarrier);
14791479
// In this case we need to add the IndirectPointerTag as additional input.
14801480
code = write_barrier_kind == kSkippedWriteBarrier
1481-
? kArchStoreIndirectWithWriteBarrier
1482-
: kArchStoreIndirectSkippedWriteBarrier;
1481+
? kArchStoreIndirectSkippedWriteBarrier
1482+
: kArchStoreIndirectWithWriteBarrier;
14831483
code |= RecordWriteModeField::encode(
14841484
RecordWriteMode::kValueIsIndirectPointer);
14851485
IndirectPointerTag tag = store.indirect_pointer_tag();

0 commit comments

Comments
 (0)