[PATCH] D32057: [AMDGPU] added SIInstrInfo::getAddNoCarry() helper

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 16:51:07 PDT 2017


arsenm added a comment.

In https://reviews.llvm.org/D32057#726726, @rampitec wrote:

> SILoadStoreOptimizer itself does not use new helper because that would require an extra SGPR and move for every combined lds operation.


What is preventing SIShrinkInstructions from handling it?



================
Comment at: lib/Target/AMDGPU/SIInstrInfo.h:771
+                                    MachineBasicBlock::iterator I,
+                                    DebugLoc DL,
+                                    unsigned DestReg) const;
----------------
const DebugLoc&


Repository:
  rL LLVM

https://reviews.llvm.org/D32057





More information about the llvm-commits mailing list