<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 9, 2014 at 11:12 AM, Artyom Skrobov <span dir="ltr"><<a href="mailto:Artyom.Skrobov@arm.com" target="_blank">Artyom.Skrobov@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
In the "Addenda to, and Errata in, the ABI for the ARM Architecture"<br>
(<a href="http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ihi0045d/index.
html" target="_blank">http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ihi0045d/index.<br>
html</a>), section 2.3.5.2 "The target-related attributes" specifies that<br>
Tag_CPU_arch_profile must have the default value of 0 when "Architecture<br>
profile is not applicable (e.g. pre v7, or cross-profile code)". LLVM<br>
violates the ABI by emitting this build attribute when targeting ARMv6M.<br>
<br>
Our proposed patch fixes this, and additionally, it amends<br>
test/MC/ARM/thumb2-mclass.s to match its apparent original purpose (to test<br>
the ARMv6M/ARMv7M commonality), and creates a new test case for the<br>
differences between ARMv6M and ARMv7M (i.e. the availability of basepri,<br>
basepri_max and faultmask registers)<br>
<br>
OK to commit?<br></blockquote><div><br></div><div>Can you split this up into two changes?  One to change/split the thumb2-mclass tests, and one for the EABI attribute conformance.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div></div>