[PATCH] D101190: [LoopIdiom] Ignore memory accesses to constant memory

Fri Apr 23 12:03:55 PDT 2021

aeubanks abandoned this revision.
aeubanks added a comment.

In D101190#2713091 <https://reviews.llvm.org/D101190#2713091>, @nikic wrote:

> While this works, I don't think this really addresses the root problem of D101017 <https://reviews.llvm.org/D101017>. Your example already folds fine with `-globals-aa -loop-idiom`. If GlobalsAA is not available at the time LoopIdiom runs, then that's a bug...

oh you're right, for some reason I thought this transform wasn't happening even with globals-aa

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101190