[PATCH] D152081: [Attributor] Add lightweight version for attribute deduction only. (WIP)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 7 12:38:49 PDT 2023


nikic added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:3791
+      &AANoAlias::ID,
+      &AANonNull::ID,
+  });
----------------
I would suggest to not include any attributes that are not inferred by FunctionAttrs (or required by Attributor internally) initially. This is something we can consider after it is enabled by default. I think it will be challenging enough without mixing it together with new attributes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152081



More information about the llvm-commits mailing list