[llvm-dev] Auto-upgrade review request

[Kaylor, Andrew via llvm-dev]

Hi all,

I've got a patch up on Phabricator (https://reviews.llvm.org/D70096) that auto-upgrades our handling of the strictfp attribute. Currently this attribute can be set on a call site independent of whether it is set on the enclosing function. We'd like to change that and have the auto-upgrader replace "dangling" uses with the nobuiltin attribute, which is functionally equivalent for such cases.

I think everyone involved with the strict FP work is in agreement about this, but none of us have much experience with the auto-upgrader and so no one has been comfortable approving the patch. I'm not sure I put the changes in the right place.

Can I get someone to look at the patch and give it a thumbs up or thumbs down?

Thanks,
Andy
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200130/cd0eeba4/attachment.html>