As discussed in https://bugs.llvm.org/show_bug.cgi?id=45951 and
D80584, the name 'tmp' is almost always a bad choice, but we have
a legacy of regression tests with that name because it was baked
into utils/update_test_checks.py.
This change makes -instnamer more consistent (already using "arg"
and "bb", the common LLVM shorthand). And it avoids the conflict
in telling users of the FileCheck script to run "-instnamer" to
create a better regression test and having that cause a warn/fail
in update_test_checks.py.
for (Instruction &I : BB)
if (!I.hasName() && !I.getType()->isVoidTy())
- I.setName("tmp");
+ I.setName("i");
}
return true;
}
define i32 @f_0(i32) {
; CHECK-LABEL: @f_0(
; CHECK: bb:
-; CHECK-NEXT: %tmp = add i32 %arg, 2
+; CHECK-NEXT: %i = add i32 %arg, 2
; CHECK-NEXT: br label %bb1
; CHECK: bb1:
-; CHECK-NEXT: ret i32 %tmp
+; CHECK-NEXT: ret i32 %i
%2 = add i32 %0, 2
br label %3