rjmccall added a comment.

Okay.  If the optimizer cannot correctly handle a mix of constrained and 
unconstrained FP operations, then the optimizer should protect itself, either 
by refusing to inline across such boundaries or by adding constraints as 
necessary before inlining.  If it does that, I don't think it's particularly 
appropriate for the frontend to warn about the combination of constrained FP 
and supposedly inlining-related attributes (which only really seems true of 
`always_inline`).  We should just mention in the documentation that this isn't 
particularly performant at the moment.

I feel quite confident that we can solve the engineering problem of efficiently 
figuring out that there's a constrained scope somewhere in a function and 
therefore we need constrained intrinsics.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69272



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

Reply via email to