arsenm marked an inline comment as done.
arsenm added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:2757
// subsequent options conflict then emit warning diagnostic.
+ // TODO: How should this interact with DenormalFP32Math?
if (HonorINFs && HonorNaNs &&
----------------
I think this should just follow along with DenormalFPMath, but I'll put this
off to the later patch since this one still is slightly awkward trying to avoid
changing the meaning of the absence of the option
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69878/new/
https://reviews.llvm.org/D69878
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
