[PATCH] D139629: clang: Stop emitting "strictfp"

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 7 11:05:59 PDT 2023


rjmccall added a comment.

In D139629#4481030 <https://reviews.llvm.org/D139629#4481030>, @zahiraam wrote:

> According to this comment https://reviews.llvm.org/D87528#2342132, it looks like @rjmccall had proposed the addition of the attribute. @rjmccall Do you recall why it was added?

My suggestion was that we track the pragma in the Clang AST with an attribute instead of a bit on the Decl.  I have no idea why IR generation is adding both a string attribute and the builtin attribute to the IR function, but if doesn't have any obvious effect, I suggest just removing it.


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

https://reviews.llvm.org/D139629



More information about the cfe-commits mailing list