<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, May 11, 2016 at 1:09 PM, Etienne Bergeron via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Author: etienneb<br>
Date: Wed May 11 15:09:17 2016<br>
New Revision: 269224<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=269224&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=269224&view=rev</a><br>
Log:<br>
[tooling] Fix missing inline keyworkd, breaking build bot.<br>
<br>
Summary:<br>
The missing keyword "inline" is causing some buildbot to fail.<br>
The symbol is not available.<br>
<br>
see: <a href="http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage/builds/2281/" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-ppc64be-linux-multistage/builds/2281/</a><br>
<br>
Reviewers: rnk<br>
<br>
Subscribers: cfe-commits, klimek<br>
<br>
Differential Revision: <a href="http://reviews.llvm.org/D20180" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20180</a><br>
<br>
Modified:<br>
    cfe/trunk/include/clang/Tooling/FixIt.h<br>
<br>
Modified: cfe/trunk/include/clang/Tooling/FixIt.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/FixIt.h?rev=269224&r1=269223&r2=269224&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/FixIt.h?rev=269224&r1=269223&r2=269224&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/include/clang/Tooling/FixIt.h (original)<br>
+++ cfe/trunk/include/clang/Tooling/FixIt.h Wed May 11 15:09:17 2016<br>
@@ -40,27 +40,27 @@ inline SourceRange getSourceRange(const<br>
<br>
 /// \brief Returns the SourceRange of an given Node. \p Node is typically a<br>
 ///        'Stmt', 'Expr' or a 'Decl'.<br>
-template <typename T> SourceRange getSourceRange(const T &Node) {<br>
+template <typename T> inline SourceRange getSourceRange(const T &Node) {<br>
   return Node.getSourceRange();<br>
 }<br>
 } // end namespace internal<br>
<br>
 // \brief Returns a textual representation of \p Node.<br>
 template <typename T><br>
-StringRef getText(const T &Node, const ASTContext &Context) {<br>
+inline StringRef getText(const T &Node, const ASTContext &Context) {<br>
   return internal::getText(internal::getSourceRange(Node), Context);<br>
 }<br>
<br>
 // \brief Returns a FixItHint to remove \p Node.<br>
 // TODO: Add support for related syntactical elements (i.e. comments, ...).<br>
-template <typename T> FixItHint createRemoval(const T &Node) {<br>
+template <typename T> inline FixItHint createRemoval(const T &Node) {<br>
   return FixItHint::CreateRemoval(internal::getSourceRange(Node));<br>
 }<br>
<br>
 // \brief Returns a FixItHint to replace \p Destination by \p Source.<br>
 template <typename D, typename S><br>
-FixItHint createReplacement(const D &Destination, const S &Source,<br>
-                            const ASTContext &Context) {<br>
+inline FixItHint createReplacement(const D &Destination, const S &Source,<br>
+                                   const ASTContext &Context) {<br>
   return FixItHint::CreateReplacement(internal::getSourceRange(Destination),<br>
                                       getText(Source, Context));<br>
 }<br></blockquote><div><br></div><div>This change doesn't make sense -- the 'inline' here is unnecessary, and unrelated to the build bot failure (which is not complaining about any of these functions). It looks like the problem is that libTooling.so isn't being linked into libclangTidyMiscModule, due to a missing dependency in clang-tools-extra/clang-tidy/misc/CMakeLists.txt.</div><div><br></div><div>Please revert this and instead add a dependency on libTooling to that CMakeLists.txt file.</div></div></div></div>