[PATCH] D49273: [InstCombine] Expand the simplification of pow() into exp2()

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 28 12:34:41 PDT 2018


evandro added a comment.

In https://reviews.llvm.org/D49273#1216110, @spatel wrote:

> Are there any negative tests where the base is not a power-of-2? If not, please add at least 1 test like that to make sure we're not transforming that.


I believe that `@test_simplify1`, `@test_simplify2`, `@test_simplify9`, `@test_simplify10`, `@test_simplify18`, `@test_simplify19` already function as negative tests for this transformation.  Yes?


https://reviews.llvm.org/D49273





More information about the llvm-commits mailing list