[PATCH] D80952: [FPEnv][Clang][Driver] Disable constrained floating point on targets lacking support.

Qing Shan Zhang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 30 19:00:15 PDT 2020


steven.zhang added inline comments.


================
Comment at: clang/lib/Basic/Targets/PPC.h:86
+
+    HasStrictFP = true;
   }
----------------
nemanjai wrote:
> I don't think we need this for now. Close is not quite there. @steven.zhang I would prefer that we initially turn this off and only flip it on once the support is complete.
> Also, is the support that is currently under development for both 32 and 64 bit architectures? If it is 64 bit only, then we can enable it only there once it is done.
Yes, it hasn't been finished yet. Agree that we can turn on it later when it is done.


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

https://reviews.llvm.org/D80952





More information about the cfe-commits mailing list