lgtm

On Mon, Jan 12, 2015 at 8:17 PM, Nico Weber <[email protected]> wrote:

> Hi,
>
> sema calls HandleVTable() with a bool parameter which is then threaded
> through three layers. The only effect of this bool is an early return at
> the last layer.
>
> Instead, remove this parameter and call HandleVTable() only if the bool is
> true. No intended behavior change, this is just a code simplification.
>
> I'm not sure why these three hops are done – it's not due to code rot,
> this was added in one piece in r103718. That's why I'm not just landing
> this but asking for precommit review – does anyone know if there's a reason
> for the current path?
>
> Nico
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to