[lld] r229030 - Remove class that really does nothing.

Chandler Carruth chandlerc at google.com
Thu Feb 12 19:14:39 PST 2015


On Thu, Feb 12, 2015 at 6:51 PM, Shankar Easwaran <shankare at codeaurora.org>
wrote:

> Hi Rui,
>
> It was a place holder to handle linker scripts. Could you revert this
> patch.
>

To relay my stance that I put in IRC: this follows the YAGNI principle. We
shouldn't keep dead, unused, untested code in tree because maybe some day
it will be needed.

This is especially true when the code is a class with *no members*. It
isn't even an interface.

If you end up needing it, add it then, with the code that needs it. Not
before.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150212/c51047e1/attachment.html>


More information about the llvm-commits mailing list