[PATCH] D58984: DAG: propagate ConsecutiveRegs flags to returns too.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 5 10:47:17 PST 2019


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:1643
 
+      bool NeedsRegBlock = TLI.functionArgumentNeedsConsecutiveRegisters(
+          I.getOperand(0)->getType(), F->getCallingConv(),
----------------
The hook name is inaccurate here. Maybe there should be an alias for it for the return value?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58984





More information about the llvm-commits mailing list