[PATCH] D69798: Implement inlining of strictfp functions

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added a comment.

In D69798#3408468 <https://reviews.llvm.org/D69798#3408468>, @andrew.w.kaylor wrote:

> I had forgotten that this patch never landed, but I was investigating a bug yesterday that I think this will help with (https://github.com/llvm/llvm-project/issues/48669).
>
> @kpn are you happy with the current form. It's gotten stale in a few places, but I think it's basically correct.

Yes, it looks fine. From reading my comments, I was worried about 'strictfp' not being attached to cloned calls. But it looks like it attaches it after the clone. So it's fine.

I'd love to see this go in the tree.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69798