aaron.ballman added a comment.

I'm also okay with this direction. I took a look to see if people seemed to be 
using this option in their build scripts (maybe we would need a louder 
deprecation period), and it seems like most of the uses out there are in forks 
of Clang. Once I excluded things that looked too clang-like, I spotted: 
https://sourcegraph.com/search?q=context:global+-file:.*test.*+-file:.*clang.*+-file:Tools.cpp+-menable-unsafe-fp-math&patternType=standard
 -- I don't have the impression we need a deprecation period for this. (Do we 
consider this to be a potentially breaking change we need to list in the 
release notes/announcements/clang-vendors?) The changes should have a release 
note, regardless of what heading we put it under.


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

https://reviews.llvm.org/D135097

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

Reply via email to