[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