[PATCH] D95748: AMDGPU: Fix dbg_value handling when forming soft clause bundles

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 1 10:33:27 PST 2021


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFormMemoryClauses.cpp:388
       RPT.reset(*B, &LiveRegsCopy);
+      DbgInstrs.clear();
+
----------------
dfukalov wrote:
> Why don't just create DbgInstrs here?
Right, it can be local to this block and cleared/destroyed after re-insertion. It will take less memory.


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

https://reviews.llvm.org/D95748



More information about the llvm-commits mailing list