[PATCH] D49617: Early exit with cheaper checks

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 20 15:26:31 PDT 2018


hiraditya added a comment.

In https://reviews.llvm.org/D49617#1170601, @davide wrote:

> In https://reviews.llvm.org/D49617#1170597, @hiraditya wrote:
>
> > In https://reviews.llvm.org/D49617#1170536, @davide wrote:
> >
> > > This is probably fine, but does it matter? i.e. you have compile time numbers?
> >
> >
> > I dont have numbers, maybe you can help with that.
>
>
> I guess my question is, what's the motivation for this change? You found this by inspection?


Yes.


https://reviews.llvm.org/D49617





More information about the llvm-commits mailing list