<div class="gmail_quote">On Mon, Jan 30, 2012 at 3:53 AM, James Molloy <span dir="ltr"><<a href="mailto:james.molloy@arm.com">james.molloy@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">









<div lang="EN-GB" link="blue" vlink="purple">

<div>

<p class="MsoNormal"><span style="color:#1f497d">Hi Chandler,<u></u><u></u></span></p>

<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>

<p class="MsoNormal"><span style="color:#1f497d">The predicate looks good,
however I’d suggest adding a few more for orthogonality while you’re there?<u></u><u></u></span></p>

<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>

<p><u></u><span style="font-family:Symbol;color:#1f497d"><span>·<span style="font:7.0pt "Times New Roman"">        
</span></span></span><u></u><span style="color:#1f497d">isArch32Bit() – this
is not necessarily !isArch64Bit() (although probably currently is for the
architectures we currently support).</span></p></div></div></blockquote><div>Yea, eventually we should have this. As Anton mentioned, we have a 16-bit arch. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-GB" link="blue" vlink="purple"><div><p><span style="color:#1f497d"><u></u><u></u></span></p>

<p><u></u><span style="font-family:Symbol;color:#1f497d"><span>·<span style="font:7.0pt "Times New Roman"">        
</span></span></span><u></u><span style="color:#1f497d">get[32,64]BitArch() –
take the current arch, and return the [32,64] bit version of it. This could be
used to massively simplify –m32/-m64 in Clang.</span></p></div></div></blockquote><div>This was going to be my next patch. =p</div></div>