[PATCH] D128163: [Coroutines] Don't add musttail call if WebAssembly are enabled
Thomas Lively via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 21 10:40:52 PDT 2022
tlively added a comment.
Thanks for taking a look at this!
================
Comment at: llvm/lib/Transforms/Coroutines/CoroSplit.cpp:1570
+ // WebAssembly engines.
+ return !Triple(M->getTargetTriple()).isWasm();
+}
----------------
It would be good if we could call out to the TargetTransformInfo to ask the backend whether it supports tail calls. Then we could gracefully handle the case where the WebAssembly `tail-call` target feature //is// enabled.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128163/new/
https://reviews.llvm.org/D128163
More information about the llvm-commits
mailing list