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

Sanjay Patel via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 3 15:43:04 PDT 2016


spatel added inline comments.


> echristo wrote in CGCall.cpp:1735
> Would be nice to get these pulled into a single fast-math string that's set and then used all over for sure. :)

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?

https://reviews.llvm.org/D24815





More information about the cfe-commits mailing list