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

[Noah Goldstein via Phabricator via llvm-commits]

goldstein.w.n added a comment.

In D148594#4279779 <https://reviews.llvm.org/D148594#4279779>, @RKSimon wrote:

> LGTM - we might want to consider a DAGISel peephole to take of cases where we lose signextended imm8 values.

DAGISel? Which file is that? X86ISelDAGToDAG.cpp?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148594