[PATCH] D100810: Use `GNUInstallDirs` to support custom installation dirs. -- LLVM

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 07:41:02 PDT 2021


lebedev.ri added a comment.

In D100810#2708650 <https://reviews.llvm.org/D100810#2708650>, @Ericson2314 wrote:

> Ah this split won't work, because the changes to `AddLLVM` and friends mean the downstream packages also need to use `GNUInstallDirs`.

So what? The suggestion to split this up was specifically to simplify review.
Nothing is stopping you from committing everything at once in the end.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100810



More information about the llvm-commits mailing list