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

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 17:15:18 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D32057#726838, @arsenm wrote:

> 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?


Good question. I will need to debug it.


Repository:
  rL LLVM

https://reviews.llvm.org/D32057





More information about the llvm-commits mailing list