Merged: [compiler] Don't elide ChangeTaggedToInt32->ChangeInt31ToTaggedSigned

LeszekSwirski · ajklein · commit f4f11c23ae2e · 2021-12-10T21:47:46.000Z
(cherry picked from commit 4fae8b1) Bug: chromium:1278387 No-Try: true No-Presubmit: true No-Tree-Checks: true Change-Id: I9b89834c094510e064988aa534ec230309996034 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3329665 Reviewed-by: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/branch-heads/9.6@{#40} Cr-Branched-From: 0b7bda0-refs/heads/9.6.180@{#1} Cr-Branched-From: 41a5a24-refs/heads/main@{#77244}
diff --git a/src/compiler/simplified-operator-reducer.cc b/src/compiler/simplified-operator-reducer.cc
@@ -77,7 +77,7 @@ Reduction SimplifiedOperatorReducer::Reduce(Node* node) {
     case IrOpcode::kChangeInt32ToTagged: {
       Int32Matcher m(node->InputAt(0));
       if (m.HasResolvedValue()) return ReplaceNumber(m.ResolvedValue());
-      if (m.IsChangeTaggedToInt32() || m.IsChangeTaggedSignedToInt32()) {
+      if (m.IsChangeTaggedSignedToInt32()) {
         return Replace(m.InputAt(0));
       }
       break;

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ Reduction SimplifiedOperatorReducer::Reduce(Node* node) {`
`77`	`77`	`case IrOpcode::kChangeInt32ToTagged: {`
`78`	`78`	`Int32Matcher m(node->InputAt(0));`
`79`	`79`	`if (m.HasResolvedValue()) return ReplaceNumber(m.ResolvedValue());`
`80`		`- if (m.IsChangeTaggedToInt32() \|\| m.IsChangeTaggedSignedToInt32()) {`
	`80`	`+ if (m.IsChangeTaggedSignedToInt32()) {`
`81`	`81`	`return Replace(m.InputAt(0));`
`82`	`82`	`}`
`83`	`83`	`break;`