<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I'm confused by the error given that getStatementsRange is a function name.  I don't have Visual Studio -- can you find a fix and send a patch? I wonder if taking the address explicitly is enough?  Or, if you know how to trigger this error in clang or gcc, I can fix it myself.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 22, 2019 at 4:31 PM Petr Penzin via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">penzn added inline comments.<br>
<br>
<br>
================<br>
Comment at: cfe/trunk/lib/Tooling/Refactoring/RangeSelector.cpp:229<br>
+RangeSelector tooling::statements(StringRef ID) {<br>
+  return RelativeSelector<CompoundStmt, getStatementsRange>(ID);<br>
+}<br>
----------------<br>
Sorry for posting here, haven't gotten my bugzilla access yet (requested though).<br>
<br>
This breaks with Visual Studio 2017 (15.7.6):<br>
<br>
RangeSelector.cpp(229): error C2971: '`anonymous-namespace'::RelativeSelector': template parameter 'Func': 'getStatementsRange': a variable with non-static storage duration cannot be used as a non-type argument<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D61774/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D61774/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D61774" rel="noreferrer" target="_blank">https://reviews.llvm.org/D61774</a><br>
<br>
<br>
<br>
</blockquote></div>