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



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

Reply via email to