Closed Bug 1112974 Opened 5 years ago Closed 5 years ago

Try to find llvm-config through clang first before looking at $PATH; r=glandium

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla37

People

(Reporter: ehsan, Assigned: ehsan)

Details

Attachments

(1 file)

Right now, if the LLVMCONFIG variable is not set in the .mozconfig, we
first look for the system default llvm-config and only then we ask clang
itself, which breaks building with the clang plugin if you make $CC and
$CXX point to a non-default clang binary.  This patch fixes the issue
by reversing the search order.
Attachment #8538228 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/4d9a7a26e09a
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Assignee: nobody → ehsan.akhgari
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.