pavelsavara wrote:
I would also love to see https://github.com/llvm/llvm-project/pull/98373
merged. Thank you.
https://github.com/llvm/llvm-project/pull/95208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
pavelsavara wrote:
could shell script named `wasm-opt` on PATH and doing nothing do the trick ?
Or even better it could write it's arguments into some file, so that they could
be asserted in the test.
2c
https://github.com/llvm/llvm-project/pull/98373
__
@@ -61,6 +61,10 @@ std::string wasm::Linker::getLinkerPath(const ArgList &Args)
const {
return ToolChain.GetProgramPath(ToolChain.getDefaultLinker());
}
+static bool IsWasip2(const llvm::Triple &TargetTriple) {
+ return TargetTriple.getOSName() == "wasip2";
---
pavelsavara wrote:
I don't know who needs to hear this, but I got bitten by `wasm-opt` being on my
PATH (while using WASI SDK 22).
I was trying to compile zlib-ng for WASIp2 and it's CMake platform detection
was passing `-O3` to `check_type_size("void *" SIZEOF_DATA_PTR)`
It worked on some mac
