[PATCH] Turn local DWARFContext helpers getFileNameForUnit() and getFileLineInfoForCompileUnit() into full-blowm DWARFDebugLine::LineTable methods.

Alexey Samsonov vonosmas at gmail.com
Mon Sep 15 13:11:57 PDT 2014


================
Comment at: lib/DebugInfo/DWARFDebugLine.cpp:663
@@ -658,3 +662,3 @@
   }
   SmallString<16> FilePath;
   uint64_t IncludeDirIndex = Entry.DirIdx;
----------------
friss wrote:
> samsonov wrote:
> > Looks like you can have a single SmallString<> to construct an absolute filepath.
> You mean removing the AbsolutePath variable in the is_relative code bellow right?
Yes.

http://reviews.llvm.org/D5354






More information about the llvm-commits mailing list