The patch looks fine, but please add a testcase.

On Wed, Sep 26, 2012 at 12:14 PM, Tijl Coosemans <[email protected]> wrote:

> Hi,
>
> The attached patch makes __builtin_bswap32 and __builtin_bswap64
> usable in constant expressions like gcc.
>
> This is my first look at clang code so perhaps review it more
> thoroughly than usual.
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to