ptomsich wrote:

> > Is this fixing a regression introduced in Clang 18? I'm wondering why the 
> > backport is needed in the first place, as this seems to be making 
> > potentially significant changes during the RC ("Make +pauth enabled in 
> > Armv8.3-a by default").
> 
> It is adding new CPU support to clang 18, specifically the ampere-1b. I'm not 
> sure what is considered acceptable at this stage. It is up to the release 
> maintainers whether they want to accept it. On a technical level I believe it 
> should be OK, but if only regressions are being fixed at this stage then it 
> might be better to wait for clang-19. I'm not sure how much @ptomsich wanted 
> to get this into clang-18?

Getting this into clang-18 is highly desirable to ensure it recognizes 
"-mcpu=ampere1b" and emits the CSSC instructions for the core.  I would prefer 
not to wait for clang-19, even though we ended up late in the cycle.

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

Reply via email to