AaronBallman wrote:

> > Also, if we're going to have a feature testing macro for this, it's 
> > premature to enable it from the command line flag -- upstream Clang doesn't 
> > yet support full bounds safety so users who would guard code on that check 
> > would be in for an unpleasant surprise.
> 
> Isn't the prefix `-fexperimental` supposed to show users that this is still 
> in development/incomplete? Maybe we can have docs or a tracking issue to see 
> the progress of the upstreaming process, so users can write the guard 
> accordingly and not have unpleasant surprises.

That doesn't help; now they're guarding based on compiler *version* information 
in their code. In general, we do not enable a feature test macro for a feature 
until we're feature complete and reasonably stable; this way users can rely on 
the feature test macro working.

https://github.com/llvm/llvm-project/pull/182994
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to