[PATCH] D127154: [AArch64] Remove isDef32

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 01:44:36 PDT 2022


dmgreen added a comment.

In D127154#3565688 <https://reviews.llvm.org/D127154#3565688>, @mstorsjo wrote:

> This triggers failed asserts for me:
> ...

Thanks for the reproducer - we are not preserving SSA in the intermediate steps, which is causing problems for subsequent combines. I feel like we should try and remove ToBeRemoved set, but that might require more testing to make sure it doesn't cause other problems.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127154



More information about the llvm-commits mailing list