On Fri, 2012-03-23 at 01:59 -0700, Christopher Li wrote: > > I just realized that this patch has the unwanted side effect that, > even if you try to define 32 bit compile in 64 bit Linux, the > __x86_64__ will > still be there. The weak define only avoid the conflict of defining > the __x86_64__ > the second time. It does not solve the case where __x86_64__ shouldn't > be there. > > The better way seems to be parsing -m32 and -m64 then set the > __x86_64__ accordingly. That reminds me we should also have a way to pass the arch triplet and have a good way to find a "default". On debian sid you need it to get all your standard includes for userspace, and we should really pass it to llvm in sparse-llvm. Cheers, Ben. -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
