[PATCH] D50246: [RISCV] Add support for computing sysroot for riscv32-unknown-elf

[Simon Cook via Phabricator via cfe-commits]

simoncook requested changes to this revision.
simoncook added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: PkmX.

I've tested this, regression tests involving linking now mostly pass as crt0 can now be found, but it seems that system headers are still being pulled in causing a couple of tests to fail. In particular using `limits.h` is causing build failures for me.

Compiling with `-v`, I see for `riscv32-unknown-elf-clang`

  #include "..." search starts here:
  #include <...> search starts here:
   /data/jenkins/workspace/riscv32-llvm-gcc-branches/install/bin/../lib/gcc/riscv32-unknown-elf/8.2.0/../../../../riscv32-unknown-elf/include
   /usr/local/include
   /data/jenkins/workspace/riscv32-llvm-gcc-branches/install/lib/clang/8.0.0/include
   /usr/include
  End of search list.

whereas for `riscv32-unknown-elf-gcc`:

  #include "..." search starts here:
  #include <...> search starts here:
   /data/jenkins/workspace/riscv32-llvm-gcc-branches/install/lib/gcc/riscv32-unknown-elf/8.2.0/include
   /data/jenkins/workspace/riscv32-llvm-gcc-branches/install/lib/gcc/riscv32-unknown-elf/8.2.0/include-fixed
   /data/jenkins/workspace/riscv32-llvm-gcc-branches/install/lib/gcc/riscv32-unknown-elf/8.2.0/../../../../riscv32-unknown-elf/include
  End of search list.

Can you see why system headers are still being pulled in unless `--sysroot` is specified?


Repository:
  rC Clang

https://reviews.llvm.org/D50246