This patch introduces basic test coverage for llvm.powi.* intrinsics.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/Analysis/CostModel/X86/powi.ll | ||
---|---|---|
8 | Probably worth adding a couple of different methods here with different i32 power factors: non-constant, negative, odd, even. big (above the limit for powi -> mul expansion): | |
llvm/test/Transforms/SLPVectorizer/X86/powi.ll | ||
8 | again - you're going to need tests with different powi factors - not just the factor = 6 case from https://github.com/llvm/llvm-project/issues/53887 - its probably satisfactory to have these tests use different factors (just make sure you rename them to state the factor)? probably worth adding at least one 'negative test' with different powi factors to make sure it doesn't try to vectorize / crash / whatever |
Probably worth adding a couple of different methods here with different i32 power factors: non-constant, negative, odd, even. big (above the limit for powi -> mul expansion):
@powi_var
@powi_6
@powi_3
@powi_n3
@powi_16