[PATCH] D142254: [X86] Transform vector SET{LE/ULT/ULE} -> SETLT and SET{GE/UGT/UGE} -> SETGT if possible

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 16 09:40:43 PST 2023


goldstein.w.n added a comment.

In D142254#4131052 <https://reviews.llvm.org/D142254#4131052>, @goldstein.w.n wrote:

> In D142254#4131023 <https://reviews.llvm.org/D142254#4131023>, @saugustine wrote:
>
>> This isn't the best reproducer, but it's as small as bugpoint can make it. Worth turning into a real test?
>>
>> ./build/bin/clang -O3 reduced.ll
>>
>> should give the failure. It does need to have assertions on.
>>
>> F26502294: reduced.ll <https://reviews.llvm.org/F26502294>
>
> Seems reasonable to add as a test. @craig.topper any objection?

Added this (after running through middle end) as a new test attached to the recommit, will push shortly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142254



More information about the llvm-commits mailing list