[PATCH] D141471: [AArch64][SVE] Fix crash for DestructiveBinaryComm zero merging

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 04:14:53 PST 2023


Allen added a comment.

In D141471#4058322 <https://reviews.llvm.org/D141471#4058322>, @paulwalker-arm wrote:

> This is what I was referring to during my review of D124325 <https://reviews.llvm.org/D124325> whereby I was suggesting your fix was more generic than you thought and could be the way to solve all such problems for all the destructive types that have two or more operands.
>
> That being said, I have no problem with build it up one destructive type at a time.

Thanks, I'll follow up to deal with other destructive types in the next step.


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

https://reviews.llvm.org/D141471



More information about the llvm-commits mailing list