[PATCH] D49617: Early exit with cheaper checks

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 20 15:17:45 PDT 2018


davide added a comment.

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?


https://reviews.llvm.org/D49617





More information about the llvm-commits mailing list