When expanding mul in type legalization make sure the type for shift amount can actually fit the value. This fixes PR30354 https://llvm.org/bugs/show_bug.cgi?id=30354.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
test/CodeGen/X86/mul-bigint.ll | ||
---|---|---|
1 ↗ | (On Diff #71064) | Call this file mul-i512.ll to match the existing filenames? Test on 32-bit targets as well if you can and use utils/update_llc_test_checks.py to generate the output |
test/CodeGen/X86/mul-bigint.ll | ||
---|---|---|
1 ↗ | (On Diff #71064) | To test on both 32-bit and 64-bit targets should I do something like: ; RUN llc < %s -mtriple=x86_64-unknown-linux-gnu ; RUN llc < %s -mtriple=x86-unknown-linux-gnu ? |
test/CodeGen/X86/mul-i512.ll | ||
---|---|---|
5 ↗ | (On Diff #71147) | Adding nounwind should cleanup the codegen a little, same for mul-i1024.ll |