[PATCH] Alignment assumptions using invariants

Chandler Carruth chandlerc at gmail.com
Thu Jul 17 12:54:50 PDT 2014


FYI, I'm OK with this, but I don't understand SCEV. Someone who does should look at it as well because my eyes juts glazed over there.

================
Comment at: lib/Transforms/Scalar/AlignmentFromInvariants.cpp:65-70
@@ +64,8 @@
+static const char aip_name[] = "Alignment from invariants";
+INITIALIZE_PASS_BEGIN(AlignmentFromInvariants, AA_NAME,
+                aip_name, false, false)
+INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
+INITIALIZE_PASS_DEPENDENCY(ScalarEvolution)
+INITIALIZE_PASS_END(AlignmentFromInvariants, AA_NAME,
+                aip_name, false, false)
+
----------------
nit: indent to "(", maybe via clang-format?

http://reviews.llvm.org/D181






More information about the llvm-commits mailing list