[PATCH] D63372: Use "willreturn" function attribute in isGuaranteedToTransferExecutionToSuccessor

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 08:00:04 PDT 2019


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

I think this looks fine with one request for change: We "decided" to use enum attributes not string attributes (for `willreturn` and others).

To seem real improvements we might need to use this function more (we will soon in the Attributor). Just to make sure, you enabled the attributor and `willreturn` deduction for your test, correct?


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

https://reviews.llvm.org/D63372





More information about the llvm-commits mailing list