[llvm-commits] [llvm] r107205 - in /llvm/trunk: bindings/ada/llvm/ docs/ include/llvm-c/ include/llvm/ include/llvm/CodeGen/ include/llvm/MC/ include/llvm/Target/ lib/AsmParser/ lib/Bitcode/Reader/ lib/Bitcode/Writer/ lib/CodeGen/ lib/CodeGen/AsmPrinter/ lib/Linker/ lib/MC/ lib/Target/ lib/Target/CppBackend/ lib/Target/XCore/AsmPrinter/ lib/Transforms/IPO/ lib/VMCore/ tools/llvm-nm/

Bill Wendling isanbard at gmail.com
Thu Jul 1 13:13:33 PDT 2010


On Jul 1, 2010, at 12:43 PM, Chris Lattner wrote:

> Please tailor this for Darwin, there is no reason to speculatively generalize this for targets that don't exist.  If a target starts existing that wants to use this, it will probably need other hooks.  Trying to anticipate this theoretical possibility just clutters the code.
> 
I can't see how this is cluttering the code. It's consistent with how the rest of the linkage code works, which isn't clutter. And there is a long tradition of wanting to anticipate for theoretical possibilities -- for example, allowing for the (unlikely) possibility of having multiple personality functions for one function with the new exception handling proposal.

I will make the change.

-bw





More information about the llvm-commits mailing list