[PATCH] D140666: [InstCombine] combine intersection for inequality icmps
Yingchi Long via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 24 05:13:37 PST 2023
inclyc added a comment.
@bcl5980
> Can we merge the code into here? It looks the logic is almost the same.
Looking at it so far, I think we have some misconceptions about merging. Some icmps may be both Mixed and NotMixed? Suggested code changes are assuming icmps are "Mixed" or not, instead of we could encounter `Mask = BMask_Mixed | BMask_NotMixed` but I think that's the fact.
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