[all-commits] [llvm/llvm-project] bfbfd8: [Driver] Linux.cpp: delete unneeded D.getVFS().exi...

Fangrui Song via All-commits all-commits at lists.llvm.org
Wed Mar 24 15:25:57 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bfbfd83f147f0e02e652f3ae2635dd834c2e6b46
      https://github.com/llvm/llvm-project/commit/bfbfd83f147f0e02e652f3ae2635dd834c2e6b46
  Author: Fangrui Song <i at maskray.me>
  Date:   2021-03-24 (Wed, 24 Mar 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/Linux.cpp
    M clang/test/Driver/linux-ld.c

  Log Message:
  -----------
  [Driver] Linux.cpp: delete unneeded D.getVFS().exists checks

Not only can this save unneeded filesystem stats, it can make `clang
--sysroot=/path/to/debian-sysroot -c a.cc` work (get `-internal-isystem
$sysroot/usr/include/x86_64-linux-gnu`) even without `lib/x86_64-linux-gnu/`.
This should make thakis happy.


  Commit: 35dd6470de847636c212d7e0cd4d7ac2995679cc
      https://github.com/llvm/llvm-project/commit/35dd6470de847636c212d7e0cd4d7ac2995679cc
  Author: Fangrui Song <i at maskray.me>
  Date:   2021-03-24 (Wed, 24 Mar 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/Linux.cpp

  Log Message:
  -----------
  [Driver] Bring back "Clean up Debian multiarch /usr/include/<triplet> madness"

This reverts commit aae84b8e3939e815bbc1e64b3b30c0f10b055be4.

The chromium goma folks want to use a Debian sysroot without
lib/x86_64-linux-gnu to perform `clang -c` but no link action. The previous
commit has removed D.getVFS().exists check to make such usage work.


Compare: https://github.com/llvm/llvm-project/compare/19435d3863e5...35dd6470de84


More information about the All-commits mailing list