[PATCH] D43449: [SelectionDAG] Add LegalTypes flag to getShiftAmountTy. Use it to unify and simplify DAGCombiner and simplifySetCC code and fix a bug.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 02:02:34 PST 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - still don't understand why we don't have a single overidable function instead of the getShiftAmountTy/getScalartShiftAmountTy duplication.


Repository:
  rL LLVM

https://reviews.llvm.org/D43449





More information about the llvm-commits mailing list