[PATCH] D34356: [PDB] Don't emit debug info associated with dead chunks

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 13:34:47 PDT 2017


ruiu added inline comments.


================
Comment at: lld/COFF/Chunks.h:183-184
+  bool isCodeView() const {
+    return SectionName.startswith(".debug") &&
+           (SectionName.size() == 6 || SectionName[6] == '$');
+  }
----------------
I'm sorry to be too picky, but isn't `SectionName == ".debug" || SectionName.startswith(".debug$")` enough? In LLD I'm really trying hard to make our code simple, boring and straightforward unless there's a strong reason to make it clever.


https://reviews.llvm.org/D34356





More information about the llvm-commits mailing list