[Lldb-commits] [PATCH] D52375: [WIP] Support multiple compile units per OSO entry in SymbolFileDWARFDebugMap

Duncan P. N. Exon Smith via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 27 14:34:43 PDT 2018

dexonsmith added inline comments.

Comment at: packages/Python/lldbsuite/test/macosx/debug_map/Makefile:22
+# Everything goes as .o files directly to the linker
aprantl wrote:
> same here, just remove it
I haven't read the full patch to see if this is relevant, but my maybe-out-of-date-Makefile-fu says this is not no-op.  It defines `C_SOURCES` as a variable, as opposed to a function-like macro (which would be `C_SOURCES =`, no colon).  IIRC, with the former, any changes to `C_SOURCES` (like `C_SOURCES += ...`) are evaluated immediately when it's a variable, lazily (on use) when it's a macro.  Usually the behaviour you want is a variable.


More information about the lldb-commits mailing list