[clang] [llvm] [SROA] Canonicalize homogeneous structs into fixed vectors (PR #165159)

Yaxun Liu via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 11 07:55:54 PDT 2026


yxsamliu wrote:

> > > I'll look at the changes tomorrow, at first glance it looks good though. Could you confirm this is the latest benchmark run? [dtcxzyw/llvm-opt-benchmark#3520](https://github.com/dtcxzyw/llvm-opt-benchmark/pull/3520). Looks like really good improvements!
> > 
> > 
> > That one is pretty close but I made some minor changes after that but forgot to trigger a new run. I just triggered a new run and let's see how it goes: [dtcxzyw/llvm-opt-benchmark#3547](https://github.com/dtcxzyw/llvm-opt-benchmark/pull/3547)
> 
> Hmm unfortunately looks like there's one regression that doesn't look good. We're vectorizing when the alloca gets promoted as a scalar anyway, but then this requires a bunch of inserts and extracts.
> 
> Sorry to be such a stickler, again I haven't reviewed much so IDK exactly what LLVM policy is, but I assume there should be a pretty high bar for transforms like SROA because it's such an important pass.

I am OK. I think we should try fixing any regressions caused by changes in llvm passes since that could be a tip of an iceberg. I will look into this. Thanks. 

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


More information about the cfe-commits mailing list