[PATCH] D100339: [Attributor] Run lightweight version of the Attributor by default.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 12 14:55:40 PDT 2021


jdoerfert added a comment.

Cool, thanks! I'll give it a swirl can you address the nit below before?



================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:2343
+      &AAAlign::ID,          &AANoCapture::ID, &AAMemoryBehavior::ID,
+      &AAMemoryLocation::ID};
+
----------------
The last two are rather costly. We should remove them for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100339



More information about the llvm-commits mailing list