]> granicus.if.org Git - llvm/commitdiff
NFC: Corrects comments that were supposed to go in with earlier commit.
authorAdrian McCarthy <amccarth@google.com>
Wed, 15 Mar 2017 20:29:06 +0000 (20:29 +0000)
committerAdrian McCarthy <amccarth@google.com>
Wed, 15 Mar 2017 20:29:06 +0000 (20:29 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@297887 91177308-0d34-0410-b5e6-96231b3b80d8

lib/DebugInfo/PDB/Native/NativeCompilandSymbol.cpp

index 24b62823eb2e216ee71f4154c7e33d6f4f83f18f..9c0cc0bf82337b6756a001cbff5c0fb72c56e9e9 100644 (file)
@@ -1,4 +1,4 @@
-//===- NativeCompilandSymbol.h - Native impl of PDBCompilandSymbol -C++ -*-===//
+//===- NativeCompilandSymbol.cpp - Native impl for compilands ---*- C++ -*-===//
 //
 //                     The LLVM Compiler Infrastructure
 //
@@ -26,9 +26,10 @@ bool NativeCompilandSymbol::isEditAndContinueEnabled() const {
 
 uint32_t NativeCompilandSymbol::getLexicalParentId() const { return 0; }
 
-// DIA, which this API was modeled after, uses counter-intuitive meanings for
-// IDiaSymbol::get_name and IDiaSymbol::get_libraryName, which is why these
-// methods may appear to be cross-mapped.
+// The usage of getObjFileName for getLibraryName and getModuleName for getName
+// may seem backwards, but it is consistent with DIA, which is what this API
+// was modeled after.  We may rename these methods later to try to eliminate
+// this potential confusion.
 
 std::string NativeCompilandSymbol::getLibraryName() const {
   return Module.Info.getObjFileName();