[PATCH] D76882: [AMDGPU] Implement CFI for non-kernel functions

Christudasan Devadasan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 14 09:11:18 PDT 2020


cdevadas added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:1276
+  OSBlock << uint8_t(dwarf::DW_OP_LLVM_offset_uconst);
+  // FIXME:
+  const unsigned SGPRByteSize = 4;
----------------
Can you write a small description of why the FIXME?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76882





More information about the llvm-commits mailing list