On 20 Aug 2009, at 07:15, Chris Lattner wrote:

I admit that the original check for darwin was completely hideous here, but this is making it a lot worse. Can this be factored out into targetInfo somehow? At the very least, can this use llvm::Triple to do this decoding?


Agreed - very ugly. The attached patch makes it much less so. Since we were already passing the Context, I've moved all of the architecture-specific tweaking inside the constructor and modified it to use llvm::Triple.

David

Attachment: clang.diff
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to