[PATCH] D140666: [InstCombine] combine intersection for inequality icmps
Yingchi Long via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 22 08:22:10 PST 2023
inclyc planned changes to this revision.
inclyc added a comment.
In D140666#4069648 <https://reviews.llvm.org/D140666#4069648>, @spatel wrote:
> In D140666#4065425 <https://reviews.llvm.org/D140666#4065425>, @inclyc wrote:
>
>> Looks like the test LLVM.Transforms/InstCombine::sign-test-and-or.ll is failed due to recent changes on this patch. I'd like to inspect it first. :)
>
> Those diffs are miscompiles if I'm seeing it correctly, so something is not right with this patch.
Thanks! Changes are planned after the Spring Festival these days. I'm sorry for the delay.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140666/new/
https://reviews.llvm.org/D140666
More information about the llvm-commits
mailing list