<div dir="ltr">And here Colin is yet another commit you have landed without review.<div><br></div><div>You seem to have been landing multiple MC changes without getting any review or comment from anyone else working on MC despite mailing them out for pre-commit review. What on earth is going on here?</div><div><br></div><div>Looking at the diff, this also seems to effectively revert multiple changes  made to the AsmParser code in the last few months. I assume you're trying to upstream patches from some internal branch, but you *really must* actually merge the trunk changes in!</div><div><br></div><div>Yet again, please revert this, and every other patch dependent on it if necessary, and every other patch you are landing outside of Hexagon without bothering to respect our code review process.</div><div><br></div><div>-Chandler</div><div><br><div class="gmail_quote"><div dir="ltr">On Sun, Nov 8, 2015 at 8:15 PM Colin LeMahieu via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">colinl abandoned this revision.<br>
colinl added a comment.<br>
<br>
252439<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14256" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14256</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div></div>