AaronBallman wrote:

> > > > @erichkeane, thank you. What's the process of including this in the 
> > > > next release?
> > > 
> > > 
> > > After CI is complete, you can click "Squash and Merge" below (if you 
> > > cannot, let us know and someone can do it for you), and it'll be included 
> > > in the 19.1 release this summer.
> > 
> > 
> > I have commit access. I want this to be part of the 18.x releases as that 
> > breaks our downstream clients.
> 
> Ah, hmm... I am not sure this qualifies for inclusion in the current release 
> branch. Perhaps @AaronBallman can comment here.

I'm not particularly comfortable putting this into 18.x; we should only be 
pushing very safe fixes to regressions and this one doesn't really qualify. 
It's debatable whether it's a regression (it sort of is, sort of isn't), but 
the changes are also relatively involved. I'd feel more comfortable with this 
in 19.x so we have more time to find and fix remaining edge cases.

https://github.com/llvm/llvm-project/pull/87281
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to