[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