[PATCH] D100063: [AMDGPU] Split GCNRegBankReassign

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 7 14:44:53 PDT 2021


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNRegBankReassign.cpp:400
 
   if (MI.isDebugValue())
     return std::make_pair(StallCycles, UsedBanks);
----------------
arsenm wrote:
> Unrelated, but this is likely a bug. This should cover all meta instructions (most likely this will do the wrong thing for kill)
Ack, will fix it separately.


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

https://reviews.llvm.org/D100063



More information about the llvm-commits mailing list