<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 18, 2017 at 10:25 PM, Davide Italiano via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">davide added inline comments.<br>
<br>
<br>
================<br>
Comment at: lib/Transforms/Utils/<wbr>MemorySSA.cpp:1915-1916<br>
 #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)<br>
-LLVM_DUMP_METHOD void MemorySSA::dump() const {<br>
-  print(dbgs());<br>
-}<br>
+LLVM_DUMP_METHOD void MemorySSA::dump() const { print(dbgs()); }<br>
 #endif<br>
<br>
----------------<br>
unrelated formatting I think (also I think the `#if` is redundant if you mark `dump()` as `LLVM_DUMP_METHOD`?)<br>
<br></blockquote><div><br></div><div>Matthias added these (though i'm to blame for clang-formatting it) in r293359</div><div>His commit message says:</div><div><br></div><div> - Public headers should just declare the dump() method but not use</div><div>      LLVM_DUMP_METHOD or #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)</div><div>    - The definition of a dump method should look like this:</div><div>      #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)</div><div>      LLVM_DUMP_METHOD void MyClass::dump() {</div><div>        // print stuff to dbgs()...</div><div>      }</div><div>      #endif </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<a href="https://reviews.llvm.org/D30154" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D30154</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>