[PATCH] D18042: Create a SymbolBody for locals

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 10 13:06:07 PST 2016


I is probably orthogonal, but having it in place is very handy to
change one thing at a time.

Note that each patch in the series makes the codebase smaller.
Unifying LocalSymbol is what I want to try in patch 4, which will
hopefully delete even more lines :-)

Cheers,
Rafael


On 10 March 2016 at 16:02, Rui Ueyama <ruiu at google.com> wrote:
> ruiu added a comment.
>
> So a natural question to ask is whether LocalSymbol is an appropriate class or not. Conceptually, being local is orthogonal to the type. There can be an undefined local symbol (and I think some ABI like PPC actually has that for some special use case.) What do you think?
>
>
> http://reviews.llvm.org/D18042
>
>
>


More information about the llvm-commits mailing list