[PATCH] D29092: PR31729: [GVNHoist] Don't hoist unsafe scalars at -Oz

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 13:07:17 PST 2017


hiraditya added a comment.

In https://reviews.llvm.org/D29092#688816, @majnemer wrote:

> In https://reviews.llvm.org/D29092#687515, @hans wrote:
>
> > Looks good to me.
> >
> > David, what do you think?
>
>
> I think we should also remove the OptForMinSize check in hostExpressions.


If you can explain why then I may be able to show why it is safe to do so.
Thanks,


https://reviews.llvm.org/D29092





More information about the llvm-commits mailing list