[PATCH] D90174: [HIP] Fix regressions due to fp contract change

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 4 08:39:45 PST 2020


yaxunl added a comment.

In D90174#2371577 <https://reviews.llvm.org/D90174#2371577>, @rjmccall wrote:

> Hmm.  Do we actually want this behavior of `fast` overriding pragmas?  What do other compilers do here?  It might be reasonable to just treat this as a bug.

I think clang is just trying to follow gcc's behavior. However, this is undesirable in certain cases. Introducing 'faststd' gives us more choices to avoid the undesirable behavior.


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

https://reviews.llvm.org/D90174



More information about the cfe-commits mailing list