[PATCH] D101421: [DebugInfo] Enable CodeView DebugInfo for basic block sections

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 2 13:59:38 PDT 2021


rnk added inline comments.


================
Comment at: llvm/lib/MC/MCStreamer.cpp:328
+  else if (FI->Section != getCurrentSectionOnly())
+    // There are multiple text sections in a function with option
+    // -basic-block-sections, creating a line table for every text secitons of a
----------------
Please make this change in the AsmPrinter, not the assembler. The assembler should still emit an error: if the user directly uses cv_loc directives, the assembler should produce an error. The compiler needs to avoid asking the assembler to do the impossible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101421



More information about the llvm-commits mailing list