[PATCH] D63390: [Codegen] TargetLowering::SimplifySetCC(): omit urem when possible

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 11:06:51 PDT 2019

Previous message: [PATCH] D63390: [Codegen] TargetLowering::SimplifySetCC(): omit urem when possible
Next message: [PATCH] D63390: [Codegen] TargetLowering::SimplifySetCC(): omit urem when possible
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

spatel accepted this revision.
spatel added a comment.

LGTM. I have no idea if it's worth the trade-off, but we could do this sooner in IR (instcombine) instead of or in addition to SDAG?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63390

Previous message: [PATCH] D63390: [Codegen] TargetLowering::SimplifySetCC(): omit urem when possible
Next message: [PATCH] D63390: [Codegen] TargetLowering::SimplifySetCC(): omit urem when possible
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list