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

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 22:06:57 PDT 2019


uenoku added a comment.

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

I'll fix it.

> 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?

Yes. I confirmed in the report that willreturn is actually deduced.


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

https://reviews.llvm.org/D63372





More information about the llvm-commits mailing list