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

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

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

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


It just does not process it. It ends up here:

  if (SDst) {
    if (SDst->getReg() != AMDGPU::VCC) {
      if (TargetRegisterInfo::isVirtualRegister(SDst->getReg()))
        MRI.setRegAllocationHint(SDst->getReg(), 0, AMDGPU::VCC);
      continue;
    }


Repository:
  rL LLVM

https://reviews.llvm.org/D32057