[PATCH] Modularizing LICM

[hfinkel at anl.gov]

REPOSITORY
  rL LLVM

================
Comment at: lib/Transforms/Scalar/LICM.cpp:75
@@ +74,3 @@
+namespace llvm {
+  struct LICMSafetyInfo
+  {
----------------
You need this definition in the header or else no other client will be able to create one of these.

================
Comment at: lib/Transforms/Scalar/LICM.cpp:76
@@ +75,3 @@
+  struct LICMSafetyInfo
+  {
+    bool MayThrow;           // The current loop contains an instruction which
----------------
Move { to the end of the previous line.

================
Comment at: lib/Transforms/Scalar/LICM.cpp:84
@@ +83,3 @@
+    {}
+  };
+}
----------------
Generically, we don't indent the contents of an anonymous namespace (this does not really matter here, because you need to move this struct to the header and not have it in an anonymous namespace).

http://reviews.llvm.org/D7291

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