<p dir="ltr">The formating still looks funny.  There are returns in the same line as ifs, which I don't think clang format produces.</p>
<div class="gmail_quote">On May 25, 2016 12:53 PM, "Petr Hosek" <<a href="mailto:phosek@chromium.org">phosek@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">phosek updated this revision to Diff 58445.<br>
phosek marked 4 inline comments as done.<br>
phosek added a comment.<br>
<br>
I've addressed issues pointed by Rafael and added FIXME comments about the location directives.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20337" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20337</a><br>
<br>
Files:<br>
  include/llvm/MC/MCObjectStreamer.h<br>
  include/llvm/MC/MCStreamer.h<br>
  lib/MC/MCAsmStreamer.cpp<br>
  lib/MC/MCObjectStreamer.cpp<br>
  lib/MC/MCParser/AsmParser.cpp<br>
  lib/MC/MCStreamer.cpp<br>
  test/MC/AsmParser/directive_fill.s<br>
  test/MC/AsmParser/directive_space.s<br>
  test/MC/AsmParser/symbolic-expression.s<br>
<br>
</blockquote></div>