Re: [PATCH v2] Makefile.llvm: simplify LLVM build

From: Nick Desaulniers
Date: Wed Apr 01 2020 - 13:48:21 EST


On Tue, Mar 31, 2020 at 11:11 PM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
>
> On Wed, Apr 1, 2020 at 3:39 AM Nick Desaulniers <ndesaulniers@xxxxxxxxxx> wrote:
> >
> > On Mon, Mar 30, 2020 at 11:25 PM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
> > >
> > > Having both LLVM_DIR and LLVM_SUFFIX seems verbose.
> >
> > I agree, so maybe just LLVM=y, and we can support both non-standard
> > locations and debian suffixes via modifications to PATH.
>
>
>
> OK, so we will start with the boolean switch 'LLVM'.
>
> People can use PATH to cope with directory path and suffixes.

Sounds good, we will modify our CI to use PATH modifications rather
than suffixes. We can even do that before such a patch to Makefile
exists.
--
Thanks,
~Nick Desaulniers