[PATCH] D59119: [SelectionDAG] Check legality for ADDCARRY in expandMUL_LOHI

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 09:04:34 PDT 2019


efriedma added a comment.

I'm not sure what you're hoping to accomplish with this change.  Even on targets where i64 ADDCARRY isn't legal, we have code to expand it (see DAGTypeLegalizer::ExpandIntRes_ADDSUBCARRY etc.), and it's still generally cheap.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59119





More information about the llvm-commits mailing list