[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