[PATCH] [LoopReroll] Alter the data structures used during reroll validation.

James Molloy james.molloy at arm.com
Thu Jan 29 13:54:48 PST 2015


Thanks Hal - committed with your changes in r227499.

Cheers,

James


REPOSITORY
  rL LLVM

================
Comment at: lib/Transforms/Scalar/LoopRerollPass.cpp:739
@@ +738,3 @@
+  // so we can iterate over the contents later in perfect order.
+  for (auto &I : *L->getHeader())
+    Uses[&I].resize(IL_End);
----------------
hfinkel wrote:
> I believe you need { } around the bodies of all range-based for loops in order to make some version of MSVC (2012?) happy.
> 
> (this applies to all of them)
Oh man, that's horrible! :(

http://reviews.llvm.org/D7260

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list