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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
