<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 October 2013 08:50, Amara Emerson <span dir="ltr"><<a href="mailto:amara.emerson@arm.com" target="_blank">amara.emerson@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  Fixed v6S-M issue and added more tests to cover v6, v6m, v6t2, v7EM, v7M.<br>
<br>
  I also noticed that the Tag_DIV_use attribute was being emitted the wrong way around. I've added some more enums to make it clearer. The ABI addenda says that DIV_use value 2 should only be used when hardware divide is available in ARM mode.<br>

<br>
<a href="http://llvm-reviews.chandlerc.com/D1811" target="_blank">http://llvm-reviews.chandlerc.com/D1811</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D1811?vs=4605&id=4661#toc" target="_blank">http://llvm-reviews.chandlerc.com/D1811?vs=4605&id=4661#toc</a><br>
<br>
Files:<br>
  lib/Target/ARM/ARMAsmPrinter.cpp<br>
  lib/Target/ARM/ARMBuildAttrs.h<br>
  test/CodeGen/ARM/2010-09-29-mc-asm-header-test.ll<br>
  test/CodeGen/ARM/2010-10-19-mc-elf-objheader.ll<br>
  test/MC/ARM/xscale-attributes.ll<br>
</blockquote></div><br></div>