[PATCH] D135462: [SelectionDAG] Do not second-guess alignment for alloca

Tue May 16 12:31:17 PDT 2023

jrtc27 added a comment.

In D135462#4347294 <https://reviews.llvm.org/D135462#4347294>, @zjaffal wrote:

> Hello this commit is causing a compiler crash for the following example:
> https://llvm.godbolt.org/z/xajYWoa8K

That seems to generate MIR where there are adjacent scaled and unscaled 32-bit stores of wzr, but AArch64LoadStoreOpt::mergeNarrowZeroStores looks pretty broken if you mix scaled-ness, as it determines IsScaled based on just one of the instructions and then uses that to determine what the immediate means. Either those need to be skipped over or the function needs to learn how to handle that mix.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135462/new/

https://reviews.llvm.org/D135462