[PATCH] D147243: [X86] MatchVectorAllZeroTest - add support for icmp(bitcast(icmp_ne(X,Y)),0) vector reduction patterns

Phoebe Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 01:56:42 PDT 2023


pengfei added a comment.

In D147243#4247943 <https://reviews.llvm.org/D147243#4247943>, @bgraur wrote:

> In D147243#4246924 <https://reviews.llvm.org/D147243#4246924>, @alexfh wrote:
>
>> In D147243#4246777 <https://reviews.llvm.org/D147243#4246777>, @RKSimon wrote:
>>
>>> Thanks @alexfh I have an idea what's going on and will deal with it tomorrow.
>>
>> Thanks! It would be nice to have a fix or revert soon, since we're blocked on this.
>
> @RKSimon could you please consider reverting as that should be the fastest and safest solution?
> You would unblock us and be able to work on the fix with no time pressure.
> Thank you in advance!

I just put a quick fix together with another issue. I'm fine with either land it or revert this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147243



More information about the llvm-commits mailing list