[all-commits] [llvm/llvm-project] 62654c: Restore functionality of --sysroot on FreeBSD afte...

Dimitry Andric via All-commits all-commits at lists.llvm.org
Sat Feb 15 16:50:17 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 62654cab7e654384ba503d6b62b1054cde19a553
      https://github.com/llvm/llvm-project/commit/62654cab7e654384ba503d6b62b1054cde19a553
  Author: Dimitry Andric <dimitry at andric.com>
  Date:   2020-02-16 (Sun, 16 Feb 2020)

  Changed paths:
    M clang/lib/Driver/ToolChains/FreeBSD.cpp
    M clang/lib/Driver/ToolChains/FreeBSD.h

  Log Message:
  -----------
  Restore functionality of --sysroot on FreeBSD after b18cb9c47

After b18cb9c47, clang would sometimes prefer the host C++ includes
(e.g. in /usr/include/c++/v1) before those specified via --sysroot.
While this behavior may be desirable on Linux, it is not so on FreeBSD,
where we make extensive use of --sysroot during the build of the base
system.  In that case, clang must *not* search outside the sysroot,
except for its own internal headers.

Add an override addLibCxxIncludePaths() to restore the old behavior,
which is to simply append /usr/include/c++/v1 to the specified sysroot.
While here, apply clang-format to the FreeBSD specific toolchain files.

Fixes PR44923.




More information about the All-commits mailing list