<div dir="ltr">Looks like I failed to dcommit in the right order, and Eric fixed it up after the fact:<div><br><div><div>$ git diff</div><div>diff --git a/tools/libclang/CIndexHigh.cpp b/tools/libclang/CIndexHigh.cpp</div><div>index d2baa13..bec0e82 100644</div><div>--- a/tools/libclang/CIndexHigh.cpp</div><div>+++ b/tools/libclang/CIndexHigh.cpp</div><div>@@ -411,7 +411,7 @@ extern "C" {</div><div><br></div><div> CXResult clang_findReferencesInFile(CXCursor cursor, CXFile file,</div><div>                                     CXCursorAndRangeVisitor visitor) {</div><div>-  LogRef Log = Logger::make(LLVM_FUNCTION_NAME);</div><div>+  LogRef Log = Logger::make(__func__);</div><div><br></div><div>   if (clang_Cursor_isNull(cursor)) {</div><div>     if (Log)</div><div>@@ -485,7 +485,7 @@ CXResult clang_findIncludesInFile(CXTranslationUnit TU, CXFile file,</div><div>     return CXResult_Invalid;</div><div>   }</div><div><br></div><div>-  LogRef Log = Logger::make(LLVM_FUNCTION_NAME);</div><div>+  LogRef Log = Logger::make(__func__);</div><div>   if (!file) {</div><div>     if (Log)</div><div>       *Log << "Null file";</div><div>diff --git a/tools/libclang/CLog.h b/tools/libclang/CLog.h</div><div>index b9309ed..e1d6a57 100644</div><div>--- a/tools/libclang/CLog.h</div><div>+++ b/tools/libclang/CLog.h</div><div>@@ -98,6 +98,6 @@ public:</div><div> /// \endcode</div><div> #define LOG_SECTION(NAME) \</div><div>     if (clang::cxindex::LogRef Log = clang::cxindex::Logger::make(NAME))</div><div>-#define LOG_FUNC_SECTION LOG_SECTION(LLVM_FUNCTION_NAME)</div><div>+#define LOG_FUNC_SECTION LOG_SECTION(__func__)</div><div><br></div><div> #endif</div><div>diff --git a/tools/libclang/CXSourceLocation.cpp b/tools/libclang/CXSourceLocation.cpp</div><div>index e75f6a7..68ecf38 100644</div><div>--- a/tools/libclang/CXSourceLocation.cpp</div><div>+++ b/tools/libclang/CXSourceLocation.cpp</div><div>@@ -131,7 +131,7 @@ CXSourceLocation clang_getLocation(CXTranslationUnit TU,</div><div>   if (line == 0 || column == 0)</div><div>     return clang_getNullLocation();</div><div><br></div><div>-  LogRef Log = Logger::make(LLVM_FUNCTION_NAME);</div><div>+  LogRef Log = Logger::make(__func__);</div><div>   ASTUnit *CXXUnit = cxtu::getASTUnit(TU);</div><div>   ASTUnit::ConcurrencyCheck Check(*CXXUnit);</div><div>   const FileEntry *File = static_cast<const FileEntry *>(file);</div><div><br></div><div>rnk@rnk4-w MINGW64 /c/src/llvm/tools/clang (master)</div><div>$ git commit -a -m 'Use __func__ instead of LLVM_FUNCTION_NAME now that all supported compilers support it'</div><div>[master 99252cd] Use __func__ instead of LLVM_FUNCTION_NAME now that all supported compilers support it</div></div><div><br></div><div>(no dcommit)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 19, 2016 at 6:25 PM, Dehao Chen <span dir="ltr"><<a href="mailto:dehao@google.com" target="_blank">dehao@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Looks like this needs to update tools/clang too?</div></blockquote></div></div></div>