[PATCH] D39440: [SimplifyCFG] Use a more generic name for the selects created by SpeculativelyExecuteBB to prevent long names from being created

Phabricator via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 12:04:37 PDT 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL317024: [SimplifyCFG] Use a more generic name for the selects created by… (authored by ctopper).

Changed prior to commit:
  https://reviews.llvm.org/D39440?vs=120918&id=121033#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D39440

Files:
  llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp


Index: llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp
===================================================================
--- llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp
+++ llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -2034,7 +2034,7 @@
     if (Invert)
       std::swap(TrueV, FalseV);
     Value *S = Builder.CreateSelect(
-        BrCond, TrueV, FalseV, TrueV->getName() + "." + FalseV->getName(), BI);
+        BrCond, TrueV, FalseV, "spec.store.select", BI);
     SpeculatedStore->setOperand(0, S);
     SpeculatedStore->applyMergedLocation(BI->getDebugLoc(),
                                          SpeculatedStore->getDebugLoc());
@@ -2069,7 +2069,7 @@
     if (Invert)
       std::swap(TrueV, FalseV);
     Value *V = Builder.CreateSelect(
-        BrCond, TrueV, FalseV, TrueV->getName() + "." + FalseV->getName(), BI);
+        BrCond, TrueV, FalseV, "spec.select", BI);
     PN->setIncomingValue(OrigI, V);
     PN->setIncomingValue(ThenI, V);
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39440.121033.patch
Type: text/x-patch
Size: 974 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171031/0c4f2393/attachment.bin>


More information about the llvm-commits mailing list