[PATCH] D35583: Debug Info: Add a file: field to DIImportedEntity
Adrian Prantl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 18 15:51:23 PDT 2017
aprantl added inline comments.
Comment at: lib/Bitcode/Reader/MetadataLoader.cpp:1684
+ HasFile ? getMDOrNull(Record) : nullptr,
+ HasFile ? Record : 0, getMDString(Record))),
> That's a nice touch.
That was the best "upgrade" I could come up with for reliably dealing with the old format. There is an assertion in DIBuilder that discourages creating new nodes with a location with a line but no file. We could upgrade that to a verifier check if we want to.
More information about the cfe-commits