[llvm] [VPlan] Track VPValue names in VPlan. (PR #81411)

via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 00:26:47 PDT 2024


================
@@ -986,8 +986,8 @@ define void @sinking_requires_duplication(ptr %addr) {
 ; CHECK-NEXT:   Successor(s): pred.store.if, pred.store.continue
 ; CHECK-EMPTY:
 ; CHECK-NEXT:   pred.store.if:
-; CHECK-NEXT:     REPLICATE ir<%gep> = getelementptr ir<%addr>, vp<[[STEPS]]>
-; CHECK-NEXT:     REPLICATE store ir<1.000000e+01>, ir<%gep>
+; CHECK-NEXT:     REPLICATE ir<%gep.1> = getelementptr ir<%addr>, vp<[[STEPS]]>
----------------
ayalz wrote:

Perhaps better to have `ir<%gep>.1`, namely, keep the original IR name inside the <>, and version a replica of it in VPlan (due to replicating and predicating), rather than version the underlying IR name.
To avoid misleading one to look for "%gep.1" in the original IR.

https://github.com/llvm/llvm-project/pull/81411


More information about the llvm-commits mailing list