[llvm] [llvm] Implement S_INLINEES debug symbol (PR #67490)

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 16:27:06 PDT 2023


================
@@ -3588,3 +3592,30 @@ void CodeViewDebug::emitDebugInfoForJumpTables(const FunctionInfo &FI) {
     endSymbolRecord(JumpTableEnd);
   }
 }
+
+void CodeViewDebug::emitInlinees(
+    const SmallSet<codeview::TypeIndex, 1> &Inlinees) {
+  // Divide the list of inlinees into chunks such that each chunk fits within
+  // one record.
+  constexpr auto ChunkSize =
+      (MaxRecordLength - sizeof(SymbolKind) - sizeof(int)) / sizeof(int);
----------------
rnk wrote:

I think `sizeof(uint32_t)` more closely expresses your intent. It's true that int is 32-bits everywhere you can run LLVM, but the reader shouldn't have to think about that.

https://github.com/llvm/llvm-project/pull/67490


More information about the llvm-commits mailing list