[PATCH] D128163: [Coroutines] Don't add musttail call if WebAssembly are enabled

[Thomas Lively via Phabricator via llvm-commits]

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