<div dir="ltr">Hi Ayrton,<div><br></div><div>Splitting in two, as Saleem mentioned, looks good to me.</div><div><br></div><div>cheers,</div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 9 January 2014 19:12, 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>_______________________________________________<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><br></div>