[llvm-commits] PATCH: Add 64-bit architecture predicate to llvm::Triple

[Chandler Carruth]

On Mon, Jan 30, 2012 at 3:53 AM, James Molloy <james.molloy at arm.com> wrote:

>  Hi Chandler,****
>
> ** **
>
> The predicate looks good, however I’d suggest adding a few more for
> orthogonality while you’re there?****
>
> ** **
>
> **·         **isArch32Bit() – this is not necessarily !isArch64Bit()
> (although probably currently is for the architectures we currently support).
>
Yea, eventually we should have this. As Anton mentioned, we have a 16-bit
arch.

> ****
>
> **·         **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.
>
This was going to be my next patch. =p
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120130/f8c5dd21/attachment.html>