[PATCH] D99439: Update @llvm.powi to handle different int sizes for the exponent

Fri May 21 06:43:15 PDT 2021

bjope marked 3 inline comments as done.
bjope added inline comments.

================
Comment at: llvm/test/Transforms/InstCombine/pow_fp_int16.ll:3

-; PR42190
+; Test case was copied from pow_fp_int.ll but adjusted for 16-bit int.
+; Assuming that we can't generate test checks for the same reason (PR42740).
----------------
xbolva00 wrote:
> Precommit? And we dont need full copy of existings tests - 2-3 tests for 16bit int are anough.
I've pre-commited the tests now. Remove some of the tests (mainly the ones related to i64. But I think most of the others are relevant as regression test to see that we get what is expected for the different scenarios also with 16-bit int.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99439