[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