[PATCH] D43002: Emit S_OBJNAME symbol in CodeView
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 10 10:53:07 PDT 2020
rnk added a comment.
Thinking about it again, maybe it should be an MCOption like David suggested. S_OBJNAME should probably refer to the final location of the native object file, which is only known to whoever sets up MCOptions. For example, with "fat" LTO, the linker is the one responsible for setting up the output file.
Named metadata nodes are things like `!llvm.ident = {...}` that you see at global scope in .ll files, but I don't think it's relevant.
As far as this patch is concerned, I don't think we should put this thing on the compile unit. The whole idea of compile units is to preserve the separation between compilation units through LTO, but we can't really do that in codeview anyway.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D43002/new/
https://reviews.llvm.org/D43002
More information about the llvm-commits
mailing list