[PATCH] D144451: [X86] Optimize umax(X,1)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

This doesn't handle i8 and i16 the same way because truncates get in the way, right? Would it be better to match this as "umax(x, 1)"  (before it becomes x86-specific instructions)? Or could we peek through a truncate and still generate the `adc` for i8/i16?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144451