[PATCH] D63379: [Attributor] Deducing existing nounwind attribute.

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 19 11:16:19 PDT 2019


sstefan1 added a comment.

In D63379#1548872 <https://reviews.llvm.org/D63379#1548872>, @sstefan1 wrote:

> > Is this sufficient to replace the old nounwind deduction? Can you disable the current one to see if there is something we cannot deduce with this one?
>
> I already tried it on some of the other tests and it worked fine. I am going to that and test it some more, and update tomorrow morning.


Sorry for the late update. I ran all tests with `-disable-nounwind-inference -attributor -attributor-disable=false` and they pass. Is that enough?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63379





More information about the llvm-commits mailing list