<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 29, 2014 at 6:40 AM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":22l" class="a3s" style="overflow:hidden">I think the right thing to do is to teach the `getVariantDir` function in `gllgo.go` to return an appropriate string (something like `"../lib${LLVM_LIBDIR_SUFFIX}[/sanitizer]"`).</div></blockquote><div><br></div><div>Why not make it getLibDir and just return "lib${LLVM_LIBDIRSUFFIX}/sanitizer"? The 'lib/../lib/.' thing seems needlessly roundabout.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":22l" class="a3s" style="overflow:hidden"> That string should eventually make its way to many of the appropriate places including the libgo build system. That and the `initCompiler` function looks like everything that needs to be taught about this.<br>
<br>
Did you end up exposing `LLVM_LIBDIR_SUFFIX` somewhere like a `#define` in a header file? If so, we should be able to pick it up from Go (see e.g. `bindings/go/llvm/version.go` for an example of how to do that).</div></blockquote></div><br><br></div></div>