[PATCH] D90524: [Driver] Enable getOSLibDir() lib32 workaround for SPARC on Linux

Mon Nov 9 05:36:00 PST 2020

ro added a comment.

As it happens, I'd arrived at exactly the same patch when I tried a build on a Debian/sparc64 system in the GCC compile farm (gcc202), submitted as D85582 <https://reviews.llvm.org/D85582>.

A couple of questions/caveats:

- I wouldn't call this patch a workaround, as you do in the summary: if `clang` cannot find the 32-bit libs, it's broken and needs to be fixed.
- Please reformat `Linux.cpp` as the pre-merge check requested.
- I'd be willing to accept this patch, but for one I'm not certain about LLVM policy about who may or may not do so.
- Besides, I believe it's a mistake to split the bug fix and the testsuite change into to different patches.  More about that in D90549 <https://reviews.llvm.org/D90549>.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90524/new/

https://reviews.llvm.org/D90524