devel/llvm-devel: fix llvm-devel.mk vars
ClosedPublic
Actions

Authored by mhorne on Dec 18 2019, 9:17 PM.

Tags

None

Referenced Files

	F106164550: D22869.id65793.diff
	Thu, Dec 26, 11:07 AM

	Unknown Object (File)
	Tue, Dec 10, 4:04 AM

	Unknown Object (File)
	Tue, Dec 3, 4:17 PM

	Unknown Object (File)
	Tue, Dec 3, 4:16 PM

	Unknown Object (File)
	Tue, Dec 3, 4:16 PM

	Unknown Object (File)
	Tue, Dec 3, 3:57 PM

	Unknown Object (File)
	Thu, Nov 28, 12:23 AM

	Unknown Object (File)
	Nov 18 2024, 5:23 PM

View All 50 Files

Subscribers

None

Details

Reviewers

brooks
pkubaj

Commits

rP520408: Update to a new snapshot.

Summary

The llvm-devel.mk file exports some variables for use with
CROSS_TOOLCHAIN. Unfortunately, the compiler variables point to
non-existent versions of clang, clang++, and clang-cpp.

Fix the substitutions in this file to properly append the -devel suffix
to these paths. For consistency, use the $LOCALBASE path for
ld.lld as well.

Diff Detail

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 28220
Build 26344: arc lint + arc unit

Event Timeline

mhorne created this revision.Dec 18 2019, 9:17 PM

Harbormaster completed remote builds in B28220: Diff 65793.Dec 18 2019, 9:17 PM

LGTM. I'll commit along with and update of the snapshot shortly

This revision is now accepted and ready to land.Dec 18 2019, 9:46 PM

In D22869#500705, @brooks wrote:

LGTM. I'll commit along with and update of the snapshot shortly

Thanks!

Closed by commit rP520408: Update to a new snapshot. (authored by brooks). · Explain WhyDec 18 2019, 11:14 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rP520408: Update to a new snapshot..

Revision Contents
Changeset List

Path

Size

devel/

llvm-devel/

Makefile

8 lines

files/

xtoolchain.mk.in

2 lines

Diff 65793

View Options

devel/llvm-devel/Makefile