diff options
author | Sanjay Patel <spatel@rotateright.com> | 2017-11-09 18:14:24 +0000 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2017-11-09 18:14:24 +0000 |
commit | 0d66010454812149603fb3676e53fb1f521e4a7d (patch) | |
tree | b473fea89f5e9e3756e18f01038c15caa66783b7 /llvm/lib/Transforms | |
parent | 8c60365d74acff80046c6981f6379368bc9826e8 (diff) | |
download | bcm5719-llvm-0d66010454812149603fb3676e53fb1f521e4a7d.tar.gz bcm5719-llvm-0d66010454812149603fb3676e53fb1f521e4a7d.zip |
[Reassociate] don't name values "tmp"; NFCI
The toxic stew of created values named 'tmp' and tests that already have
values named 'tmp' and CHECK lines looking for values named 'tmp' causes
bad things to happen in our test line auto-generation scripts because it
wants to use 'TMP' as a prefix for unnamed values. Use less 'tmp' to
avoid that.
llvm-svn: 317818
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r-- | llvm/lib/Transforms/Scalar/Reassociate.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/Reassociate.cpp b/llvm/lib/Transforms/Scalar/Reassociate.cpp index 1f32f9f24aa..90b13ab1730 100644 --- a/llvm/lib/Transforms/Scalar/Reassociate.cpp +++ b/llvm/lib/Transforms/Scalar/Reassociate.cpp @@ -997,7 +997,7 @@ static Value *EmitAddTreeOfValues(Instruction *I, Value *V1 = Ops.back(); Ops.pop_back(); Value *V2 = EmitAddTreeOfValues(I, Ops); - return CreateAdd(V2, V1, "tmp", I, I); + return CreateAdd(V2, V1, "reass.add", I, I); } /// If V is an expression tree that is a multiplication sequence, @@ -1604,7 +1604,7 @@ Value *ReassociatePass::OptimizeAdd(Instruction *I, RedoInsts.insert(VI); // Create the multiply. - Instruction *V2 = CreateMul(V, MaxOccVal, "tmp", I, I); + Instruction *V2 = CreateMul(V, MaxOccVal, "reass.mul", I, I); // Rerun associate on the multiply in case the inner expression turned into // a multiply. We want to make sure that we keep things in canonical form. |