[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path and include/x86_64-linux-gnu/c++/v1 libc++ path

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 11 11:21:52 PDT 2021


phosek added a comment.

In D110663#3055813 <https://reviews.llvm.org/D110663#3055813>, @phosek wrote:

> I'm still worried that this solution as implemented is going to break existing users, including major one like Rust.
>
> I'm not also not a fan of special casing individual targets like Fuchsia directly inside `Driver`, I believe that any per-target logic should live in the corresponding `ToolChain` subclass.

Given the potential wide ranging impact of this change, it might be also worth starting a thread on llvm-dev to reach a broader audience.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110663



More information about the cfe-commits mailing list