[llvm-branch-commits] [llvm] release/19.x: [AArch64] Add streaming-mode stack hazard optimization remarks (#101695) (PR #102168)

[Aaron Ballman via llvm-branch-commits]

AaronBallman wrote:

> The patch here is pretty big in size, but it seems to only affects the remarks, on the other hand it doesn't seem to really fix anything and in that case I feel like RC3 might be the wrong time to merge this. Is there a huge upside to take this this late in the process?
> 
> Also ping @jroelofs as aarch64 domain expert and @AaronBallman as clang maintainer.

We had 8 release candidates for 18.x and I would *very much* like to avoid that happening again, so I think that because we're about to hit rc3 (the last scheduled rc before we release according to the release schedule posted at https://llvm.org/) we should only be taking low-risk, high-impact changes such as fixes to regressions or obviously correct changes. I don't think this patch qualifies; is there significant risk to not putting this in? (e.g., does this fix what you would consider to be a stop-ship level issue of some kind?)

https://github.com/llvm/llvm-project/pull/102168