rjmccall added a comment.

> Well, just like for all the other FP builder methods, you can use the 
> setIsFPConstrained method on the builder object to switch between strict and 
> non-strict mode. Does this not suffice, or is there anything particular about 
> the comparisons that would require anything extra?

Ah, sorry, I forgot that `IsFPConstrained` is about whether we emit the 
intrinsics at all and not whether the intrinsics are currently recording real 
constraints.

I think I have a slight preference for the second option, where there's a 
single method that does all the work for the two cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71467



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

Reply via email to