[llvm] r334543 - Fix how LLVMOPTIONALCOMPONENTS is passed to llvm-build

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 14:37:35 PDT 2018


On Wed, Jun 13, 2018 at 2:33 PM Ahmed Bougacha <ahmed.bougacha at gmail.com>
wrote:

> On Wed, Jun 13, 2018 at 11:56 AM, Reid Kleckner <rnk at google.com> wrote:
> > Thanks for the revert. I agree, it did look quoted correctly, but I
> figured
> > this was a workaround for some platform with busted quoting rules.
>
> No worries;  the author replied elsewhere: it turned out to be some
> python configuration issue.
>
> > I don't see why passing an empty argument enables building the intel jit
> > events library, but I have no interest in investigating it.
>
> " " evaluates as true in cmake
>

Yes, but the llvm-build tool is python, and we are not checking
LLVMOPTIONALCOMPONENTS anywhere for truth, so far as I can see.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180613/95bdcb2a/attachment.html>


More information about the llvm-commits mailing list