search for: aliasatom

...present. My idea was to fix the problem before it gets bigger. My proposal would be to move Simple.h and Alias.h to Core. Similar to UndefinedAtom.h etc. It would be even nicer to make the naming consistent as well, since there already is UndefinedAtom.h SharedLibaryAtom.h etc Maybe: SimpleAtom.h AliasAtom.h Moving the file to core fixes the dependencies. On Mon, Jun 2, 2014 at 9:25 AM, Rui Ueyama <ruiu at google.com> wrote: > On Mon, Jun 2, 2014 at 12:01 AM, Bas van den Berg < > b.van.den.berg.nl at gmail.com> wrote: > >> Hi, >> >> I've been following...

...ove Simple.h and Alias.h to Core. Similar to >> UndefinedAtom.h etc. >> It would be even nicer to make the naming consistent as well, since there >> already is >> UndefinedAtom.h >> SharedLibaryAtom.h >> etc >> >> Maybe: >> SimpleAtom.h >> AliasAtom.h >> >> Moving the file to core fixes the dependencies. >> >> >> >> >> On Mon, Jun 2, 2014 at 9:25 AM, Rui Ueyama <ruiu at google.com> wrote: >>> >>> On Mon, Jun 2, 2014 at 12:01 AM, Bas van den Berg >>> <b.van.den.berg.n...

Hi, I've been following the changes in LLD's software architecture. Recently a new file was added: Alias.h In this file, the curret Atom set is extended with an AliasAtom. While the change seems innocent enough, it has some nasty potential. Simple.h and Alias.h are placed in the ReaderWriter component. This is the 2nd component in the layering: Driver ReaderWriter Passes Core The problem is that Core depends on these files. So in my opinion, they belong to that Com...

...t; It would be even nicer to make the naming consistent as well, since >> there >> >> already is >> >> UndefinedAtom.h >> >> SharedLibaryAtom.h >> >> etc >> >> >> >> Maybe: >> >> SimpleAtom.h >> >> AliasAtom.h >> >> >> >> Moving the file to core fixes the dependencies. >> >> >> >> >> >> >> >> >> >> On Mon, Jun 2, 2014 at 9:25 AM, Rui Ueyama <ruiu at google.com> wrote: >> >>> >> >>> On...