-
Notifications
You must be signed in to change notification settings - Fork 568
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[LLVMCPU] RISC-V failed to build int8 MobileNetV2 and EfficientNet #15038
Comments
To reproduce: Download the MLIR: https://storage.googleapis.com/iree-github-actions-postsubmit-artifacts/6303857967/1/e2e-test-artifacts/iree_MobileNetV2_int8_tflite_.mlir Run: iree-compile --output-format=vm-bytecode --mlir-print-op-on-diagnostic=false --iree-hal-target-backends=llvm-cpu --iree-input-type=tosa --iree-llvmcpu-target-triple=riscv32-pc-linux-gnu --iree-llvmcpu-target-cpu=generic-rv32 --iree-llvmcpu-target-abi=ilp32 --iree-llvmcpu-target-cpu-features=+m,+a,+f,+zvl512b,+zve32f --riscv-v-fixed-length-vector-lmul-max=8 iree_MobileNetV2_int8_tflite_.mlir -o module.vmfb |
stellaraccident
added a commit
that referenced
this issue
Sep 26, 2023
Landing with a regression on the RISC-V backend: #15038 --------- Co-authored-by: Jerry Wu <cheyuw@google.com>
This patch may fix it
|
lukel97
added a commit
to lukel97/llvm-project
that referenced
this issue
Sep 27, 2023
…def at 0 This fixes a crash seen in iree-org/iree#15038 and elsewhere. We were reducing the LMUL for inserts into undef at 0 without inserting it back into the original LMUL at the end. But we don't actually perform the slidedown in this path, so we can just skip reducing LMUL here.
lukel97
added a commit
to llvm/llvm-project
that referenced
this issue
Sep 28, 2023
…def at 0 (#67535) This fixes a crash seen in iree-org/iree#15038 and elsewhere. We were reducing the LMUL for inserts into undef at 0 without inserting it back into the original LMUL at the end. But we don't actually perform the slidedown in this path, so we can just skip reducing LMUL here.
A fix was landed in llvm/llvm-project@b14f6ee, does that fix the crash here? |
legrosbuffle
pushed a commit
to legrosbuffle/llvm-project
that referenced
this issue
Sep 29, 2023
…def at 0 (llvm#67535) This fixes a crash seen in iree-org/iree#15038 and elsewhere. We were reducing the LMUL for inserts into undef at 0 without inserting it back into the original LMUL at the end. But we don't actually perform the slidedown in this path, so we can just skip reducing LMUL here.
Confirmed the issue has been fixed, thanks! |
pzread
pushed a commit
that referenced
this issue
Oct 3, 2023
As #15038 (comment), the crashes on RISC-V benchmark targets have been fixed. Fix #15038
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With the LLVM integration #15020, RISC-V failed to build these targets with error:
The text was updated successfully, but these errors were encountered: