[PATCH] D158529: [InferAlignment] Implement InferAlignmentPass

[Danila Malyutin via Phabricator via llvm-commits]

danilaml added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1150
+  if (EnableInferAlignmentPass)
+    FPM.addPass(InferAlignmentPass());
   if (IsFullLTO) {
----------------
nikic wrote:
> danilaml wrote:
> > I was wondering why the pass was added after LoopeVectorize pass and not before? Wouldn't LV benefit from knowing that some pointers are better aligned? Or it doesn't care about this at the moment?
> LV uses getOrEnforceKnownAlignment(), so it shouldn't matter. Did you see any issues relating to this?
> 
> (This is true pretty generally -- the passes that care about alignment will do their own inference.)
I haven't seen the issues with pass buidler pipeline, no. I just use custom pipeline and got caught a bit off guard by this change, so I'm now in the process of figuring out where to insert this new pass ;P


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158529/new/

https://reviews.llvm.org/D158529