[PATCH] Add methods to get archive file name from member file
Shankar Kalpathi Easwaran
shankarke at gmail.com
Thu Feb 5 14:33:36 PST 2015
================
Comment at: include/lld/Core/File.h:65
@@ -62,1 +64,3 @@
+ return *new (_allocator) std::string(
+ (_archivePath + "(" + _path + ")").str());
}
----------------
ruiu wrote:
> shankarke wrote:
> > Could we cache this path.
> We could, but the cache needs to be invalidated when _archivePath is updated. This is used only for logging and debugging, I think we don't need to do that.
I dont think we ever update the archivePath after setting it.
================
Comment at: lib/ReaderWriter/FileArchive.cpp:206
@@ -205,3 +205,3 @@
result = std::move(files[0]);
if (std::error_code ec = result->parse())
return ec;
----------------
There is still an issue that if there was a parse error, diagnostics will not show the proper error. Say the file was of a different architecture. I remember this was the reason I created ArchiveMemoryBuffer.
http://reviews.llvm.org/D7447
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list