[PATCH] Rename BlackList class to GlobalList and move it to Transforms/Utils.

Alexey Samsonov samsonov at google.com
Tue Jul 9 11:55:59 PDT 2013


On Tue, Jul 9, 2013 at 9:01 PM, Peter Collingbourne <peter at pcc.me.uk> wrote:

> On Tue, Jul 09, 2013 at 11:02:43AM +0400, Alexander Potapenko wrote:
> > Are these lists always applied to GlobalValue descendants?
>
> Yes.
>
> > Anyway,
> > "global-something" is poor because it reflects neither what we do with
> > the globals, nor which globals are put into the list (obviously not
> > all of them).
> > How about SpecialCaseList, since we're telling the compiler which
> > objects are to be handled in a non-default way?
>
> I can live with that name.
>

I'm ok with the rest of this patch if we agree on this name.


>
> Thanks,
> --
> Peter
>



-- 
Alexey Samsonov, MSK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130709/99d418f4/attachment.html>


More information about the llvm-commits mailing list