[PATCH] D44054: [DWARFv5] Emit file 0 to the line table.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 2 17:01:37 PST 2018


probinson added inline comments.


================
Comment at: llvm/lib/MC/MCAsmStreamer.cpp:1119
+    MD5::MD5Result *Checksum, Optional<StringRef> Source, unsigned CUID) {
+  assert(CUID == 0);
+
----------------
aprantl wrote:
> That looks odd — is that a placeholder for later?
No, it's because asm output doesn't know how to distinguish multiple compile units in the line table.  The corresponding MCStreamer (direct-to-object) method can handle different CUIDs.


https://reviews.llvm.org/D44054





More information about the llvm-commits mailing list