[PATCH] D157894: InstSimplify: Handle exp10(log10(x)) -> x

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D157894#4585957 <https://reviews.llvm.org/D157894#4585957>, @kpn wrote:

> How do we know that x is >0?
>
> But I see that we don't worry about this for the existing cases so LGTM.

I guess you can argue that reassoc grants ignoring it, but it probably should require nnan


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157894/new/

https://reviews.llvm.org/D157894