kpn added a comment.

In D62731#1750427 <https://reviews.llvm.org/D62731#1750427>, @mibintc wrote:

> In D62731#1750412 <https://reviews.llvm.org/D62731#1750412>, @kpn wrote:
>
> > Does anyone think a warning is appropriate because the new flags are 
> > exercising experimental, incomplete code in both clang and llvm? The 
> > warning would be removed when we believe the feature is complete and ready 
> > to use.
>
>
> @kpn Can you say more about "incomplete code in ... clang".  I don't know 
> what's missing from clang.


See D70256 <https://reviews.llvm.org/D70256>. Calls to clang's math builtins 
that are llvm intrinsics need to be changed to be calls to the constrained 
intrinsics. I've been waiting to submit the patches because they weren't 
testable without these command line options.

There may be other issues as well. I'm not sure.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62731



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to