[PATCH] MachO-embedded: refactor Clang to use "*-*-unknown-macho" triple
Rafael Ávila de Espíndola
rafael.espindola at gmail.com
Mon Jan 6 14:04:42 PST 2014
The patch is OK by me.
It is all MachO specific, so I would suggest getting someone at Apple to take another look. Another thing you might want to do is to split the patch a bit to make it easier to read. Some possible sub patches.
* Rename DarwinTool to MachOTool.
* Update the method names to start with a lowercase letter.
*
================
Comment at: lib/Driver/ToolChains.h:214
@@ +213,3 @@
+ /// Add the linker arguments to link the ARC runtime library.
+ virtual void addLinkARCArgs(const llvm::opt::ArgList &Args,
+ llvm::opt::ArgStringList &CmdArgs) const {}
----------------
nit: Usually there is no advantage in having inline virtual methods (other than the destructor) in non final classes.
http://llvm-reviews.chandlerc.com/D2416
More information about the cfe-commits
mailing list