I can't check the code at the moment but somewhere in Error.h it specifically recommends using the explicit `Error::success` syntax<br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 15, 2016 at 3:23 PM Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu accepted this revision.<br>
ruiu added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM<br>
<br>
<br>
================<br>
Comment at: lib/DebugInfo/CodeView/CVTypeVisitor.cpp:1<br>
@@ +1,2 @@<br>
+//===- CVTypeVisitor.cpp ----------------------------------------*- C++ -*-===//<br>
+//<br>
----------------<br>
Remove `-*- C++ -*-` since this file has .cpp extnesion.<br>
<br>
================<br>
Comment at: lib/DebugInfo/CodeView/CVTypeVisitor.cpp:21<br>
@@ +20,3 @@<br>
+  Data = Data.drop_front(sizeof(*Res));<br>
+  return Error::success();<br>
+}<br>
----------------<br>
This is not directly related to this change, but should we write `return {}` instead of `return Error::success()` for the sake of brevity?<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21410" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21410</a><br>
<br>
<br>
<br>
</blockquote></div>