[PATCH] D24815: [clang] make reciprocal estimate codegen a function attribute

Eric Christopher via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 3 16:46:14 PDT 2016


echristo added inline comments.


> spatel wrote in CGCall.cpp:1735
> I'm probably not imagining some use case, but I was hoping that we can just delete the 4 (fast/inf/nan/nsz) that are already covered by instruction-level FMF. An auto-upgrade might be needed within LLVM...and/or a big pile of regression test changes?

No, that seems reasonable. I think it makes more sense as a follow-on patch though.

https://reviews.llvm.org/D24815





More information about the cfe-commits mailing list