On Wed, Oct 23, 2013 at 10:28 AM, Rafael Espíndola <
[email protected]> wrote:

> On 23 October 2013 12:48, Rafael Espíndola <[email protected]>
> wrote:
> > I am having second thoughts about this patch. IsUsed is a common
> > operation. We probably don't want to loop in it. We could loop in
> > setUsed to set every decl used, but we still have the problem of what
> > to do about UsedAttr.
> >
> > In the end having the callers be careful might be the best, but
> > brittle, solution. I will see if I can fix the small bug without
> > introducing a loop in isUsed.
>
> OK. Attached is a patch that fixes the bug and nothing else. That way
> we should be able to discuss the refactoring in isolation.


Sounds good, go ahead.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to