<div id="geary-body" dir="auto"><div><br></div></div><div id="geary-quote" dir="auto"><br>On Fri, Sep 8 2023 at 10:36:48 AM +02:00:00, Fabian Grünbichler <fabian@gruenbichler.email> wrote:<br><blockquote type="cite"><div class="plaintext"><blockquote style="white-space: break-spaces;"> On Tue, Aug 8 2023 at 06:22:28 PM -04:00:00, Andres Salomon</blockquote><blockquote style="white-space: break-spaces;"></blockquote></div></blockquote><span style="white-space: break-spaces;">[...]<br></span><blockquote type="cite"><div class="plaintext"><blockquote style="white-space: break-spaces;"> 
 This patch adds "cargo:rustc-link-search=/usr/lib/clang/15/lib/linux/" and
 "cargo:rustc-link-lib=static=clang_rt.profile-x86_64" (or whatever
 architecture it happens to be building on) to the profiler_builtins build
 flags when the target matches the host. In the case where the target is
 different from the host, it assumes a wasm build and sets
 "rustc-link-lib=static=clang_rt.builtins-wasm32" or
 "cargo:rustc-link-lib=static=clang_rt.builtins-wasm64" depending upon
 whether the target is 32 or 64 bits, respectively.
</blockquote><span style="white-space: break-spaces;">
Thanks for the patch! the Windows part in d/rules looks wrong to me -
that is built with mingw, not wasi. I assume you are not interested in
that part, so maybe it would also be an option to guard it based on
target and just build it for the regular one (or regular+wasm, but not
windows/mingw)?
</span></div></blockquote><span style="white-space: break-spaces;"><div><span style="white-space: break-spaces;"><br></span></div>Yeah, I don't really care about that part - I was just trying to guess about what was 'correct'.</span><div><span style="white-space: break-spaces;"><br></span></div></div>