SCCCaptured is trivially false on entry to this loop and not modified inside it.
authorNick Lewycky <nicholas@mxc.ca>
Thu, 5 Jan 2012 22:21:45 +0000 (22:21 +0000)
committerNick Lewycky <nicholas@mxc.ca>
Thu, 5 Jan 2012 22:21:45 +0000 (22:21 +0000)
Eliminate the dead test for it on each loop iteration. No functionality change.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@147616 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/IPO/FunctionAttrs.cpp

index 9e30c40e20c4a6f8dbdf072519a1675c4fb483c9..f3f622843340f8a0caf13468a5b73072af1f9295 100644 (file)
@@ -461,7 +461,7 @@ bool FunctionAttrs::AddNoCaptureAttrs(const CallGraphSCC &SCC) {
     }
     if (SCCCaptured) continue;
 
-    for (unsigned i = 0, e = ArgumentSCC.size(); i != e && !SCCCaptured; ++i) {
+    for (unsigned i = 0, e = ArgumentSCC.size(); i != e; ++i) {
       Argument *A = ArgumentSCC[i]->Definition;
       A->addAttr(Attribute::NoCapture);
       ++NumNoCapture;