[LLVMdev] [lld][RFC] TargetLayout class removing

[Rui Ueyama]

I'm wondering if we even need TargetLayout.

DefaultLayout is the only derived class of Layout. TargetLayout is the only
derived class of DefaultLayout. It looks like we can merge Layout,
DefaultLayout and TargetLayout into one.

On Fri, Apr 3, 2015 at 12:36 PM, Simon Atanasyan <simon at atanasyan.com>
wrote:

> Hi Rui, Shankar,
>
> Do we really need empty TargetLayout class? No we have the following
> hierarchy:
>
> Layout <- DefaultLayout<> <- TargetLayout<> <- xxxTargetLayout<>
>
> I suggest to remove TargetLayout and rename DefaultLayout to TargetLayout.
>
> Thoughts?
>
> --
> Simon Atanasyan
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20150403/9703c05b/attachment.html>