Skip to content

Commit

Permalink
Merged: [compiler] StackCheck can have side effects
Browse files Browse the repository at this point in the history
Bug: chromium:1452137
(cherry picked from commit e548943)

Change-Id: I4be8b53f5668839515efefa566476d8e9c6182dd
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4637186
Auto-Submit: Tobias Tebbi <tebbi@chromium.org>
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Reviewed-by: Toon Verwaest <verwaest@chromium.org>
Commit-Queue: Toon Verwaest <verwaest@chromium.org>
Cr-Commit-Position: refs/branch-heads/11.5@{v8#22}
Cr-Branched-From: 0c4044b-refs/heads/11.5.150@{#1}
Cr-Branched-From: b71d303-refs/heads/main@{#87781}
  • Loading branch information
tebbi authored and V8 LUCI CQ committed Jun 23, 2023
1 parent 5865a90 commit 93cc4b4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/compiler/js-operator.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1403,7 +1403,7 @@ const Operator* JSOperatorBuilder::CloneObject(FeedbackSource const& feedback,
const Operator* JSOperatorBuilder::StackCheck(StackCheckKind kind) {
return zone()->New<Operator1<StackCheckKind>>( // --
IrOpcode::kJSStackCheck, // opcode
Operator::kNoWrite, // properties
Operator::kNoProperties, // properties
"JSStackCheck", // name
0, 1, 1, 0, 1, 2, // counts
kind); // parameter
Expand Down

0 comments on commit 93cc4b4

Please sign in to comment.