Skip to content

Conversation

@penzn
Copy link
Contributor

@penzn penzn commented May 21, 2022

Temporary disable LoopUnswitch in apply_func_passes(), based on LLVM version. Need to reenable when binding for new loop unswitch pass gets added or the binding is implemented internally.

@penzn penzn changed the base branch from main to dev/upgrade_llvm_wasi_sdk_emsdk May 23, 2022 16:35
@penzn
Copy link
Contributor Author

penzn commented May 23, 2022

Now against a better branch :)

@penzn penzn force-pushed the llvm-loop-unswitch-pass branch from c5f11bb to 4d8a6e6 Compare May 25, 2022 06:30
Disable LoopUnswitch pass for LLVM 15 - C binding has been removed and binding
for new pass hasn't been added. Need to reenable when binding gets added to
LLVM or when we implement it in WAMR.
@penzn penzn force-pushed the llvm-loop-unswitch-pass branch from 4d8a6e6 to e1443e5 Compare June 8, 2022 17:38
@penzn penzn changed the title Use new LoopUnswitch pass Temporarily disable LoopUnswitch pass Jun 8, 2022
@penzn penzn marked this pull request as ready for review June 8, 2022 17:43
@penzn
Copy link
Contributor Author

penzn commented Jun 8, 2022

I've proposed adding the binding: https://reviews.llvm.org/D127323, if that effort falls through, I'll add it internally.

@wenyongh wenyongh merged commit 9d2d2f0 into bytecodealliance:dev/upgrade_llvm_wasi_sdk_emsdk Jun 9, 2022
wenyongh added a commit to wenyongh/wasm-micro-runtime that referenced this pull request Jun 9, 2022
wenyongh pushed a commit that referenced this pull request Jul 29, 2022
Since legacy binding for loop unswitch pass was removed and we can't get
it back. Implement its equivalent in `aot_llvm_extra.cpp` and use it in
`aot_compiler.c`.

Follow up to #1183.
loganek pushed a commit to loganek/wasm-micro-runtime that referenced this pull request Aug 31, 2022
Since legacy binding for loop unswitch pass was removed and we can't get
it back. Implement its equivalent in `aot_llvm_extra.cpp` and use it in
`aot_compiler.c`.

Follow up to bytecodealliance#1183.
vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
Since legacy binding for loop unswitch pass was removed and we can't get
it back. Implement its equivalent in `aot_llvm_extra.cpp` and use it in
`aot_compiler.c`.

Follow up to bytecodealliance#1183.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants