[cfe-commits] r140520 - /cfe/trunk/include/clang/Basic/SourceManager.h

[Argyrios Kyrtzidis]

Author: akirtzidis
Date: Mon Sep 26 03:01:45 2011
New Revision: 140520

URL: http://llvm.org/viewvc/llvm-project?rev=140520&view=rev
Log:
Remove SourceManager::getLocation().

It's not descriptive enough and it's just a call of translateFileLineCol()
followed by a call to getMacroArgExpandedLocation(), which is better to be
called explicitly since it has a certain cost and is not always required.

Modified:
    cfe/trunk/include/clang/Basic/SourceManager.h

Modified: cfe/trunk/include/clang/Basic/SourceManager.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/SourceManager.h?rev=140520&r1=140519&r2=140520&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/SourceManager.h (original)
+++ cfe/trunk/include/clang/Basic/SourceManager.h Mon Sep 26 03:01:45 2011
@@ -1107,20 +1107,6 @@
   ///
   /// If the source file is included multiple times, the source location will
   /// be based upon the first inclusion.
-  ///
-  /// If the location points inside a function macro argument, the returned
-  /// location will be the macro location in which the argument was expanded.
-  /// \sa getMacroArgExpandedLocation
-  SourceLocation getLocation(const FileEntry *SourceFile,
-                             unsigned Line, unsigned Col) {
-    SourceLocation Loc = translateFileLineCol(SourceFile, Line, Col);
-    return getMacroArgExpandedLocation(Loc);
-  }
-
-  /// \brief Get the source location for the given file:line:col triplet.
-  ///
-  /// If the source file is included multiple times, the source location will
-  /// be based upon the first inclusion.
   SourceLocation translateFileLineCol(const FileEntry *SourceFile,
                                       unsigned Line, unsigned Col) const;