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

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:2338
 
+  DenseSet<const char *> AllowedAAs = {
+      &AAReturnedValues::ID, &AANoUnwind::ID, &AANoSync::ID,
----------------
madhur13490 wrote:
> madhur13490 wrote:
> > May I know the rationale behind this list? Is it  result of some empirical analysis?
> Is there any plan to allow backends to customize this list?
These are relatively simple to compute attributes that pretty much match the FunctAttrs pass. It's an initial list, not set in stone.

Customization hooks are welcome as patches once something lands :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100339