royjacobson added a comment.

In D134898#3825255 <https://reviews.llvm.org/D134898#3825255>, @cor3ntin wrote:

> Thanks for catching that, I missed there was a feature test macro for it.
> It doesn't really make sense for it to be a feature test macro (given WG21's 
> current policies) so I'll make raise an issue about that but I'm happy to 
> approve the change in the meantime

Then I hope we won't have to revert this in two years or something :)

But in the meantime I agree this is what users will expect so it's better to 
define it than not to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134898

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

Reply via email to