[PATCH] D148594: [X86] Transform `(cmp eq/ne trunc(X), C)` -> `(cmp eq/ne X, Zext(C))`

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 18 11:43:46 PDT 2023


goldstein.w.n marked an inline comment as done.
goldstein.w.n added a comment.

In D148594#4276834 <https://reviews.llvm.org/D148594#4276834>, @pengfei wrote:

> Should add non-zero tests?

It felt like we saw enough changes throughout `CodeGen/X86/` that it was sufficient. But if you prefer
I can add some new explicit tests for it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148594



More information about the llvm-commits mailing list