<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 8, 2013 at 1:03 PM, Derek Schuff <span dir="ltr"><<a href="mailto:dschuff@google.com" target="_blank">dschuff@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think it might be better to just throw an error on regparm<br>
attributes (i.e. by setting regParmMax=0 in PNaClTargetInfo) rather<br>
than ignoring them. That way if someone tries to build against glibc<br>
headers (the reason for this code's existence in the first place), it<br>
will fail loudly rather than just silently generating incompatible<br>
code.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div style>Good point. Attaching updated patch. PNaClTargetInfo now sets RegParmMax to 0 explicitly to disallow it, and NaClTargetInfo inherits it from the underlying architecture.</div>
<div style><br></div><div style>As for tests, I noticed that with the removal of regparm support, le32-arguments.c already covers the interesting cases so what remains for le32-regparm.c is to verify the Clang error.</div>
<div style><br></div><div style>Eli</div><div style><br></div><div style><br></div><div><br></div></div></div></div>