<p dir="ltr">Lgtm. Just be sure to run check-all before committing :-)</p>
<div class="gmail_quote">On May 27, 2016 1:53 AM, "Petr Hosek" <<a href="mailto:phosek@chromium.org">phosek@chromium.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">phosek retitled this revision from "[MC] Use obj output for .fill tests where needed" to "[MC] Support symbolic expressions in assembly directives".<br>
phosek updated the summary for this revision.<br>
phosek updated this revision to Diff 58756.<br>
phosek added a comment.<br>
<br>
The original change was reverted because of the test failure, so I've updated this revision to include both the original change and the updated test which is now passing. Is this okay?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20656" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20656</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>