Bug 212312 - devel/llvm38 and later fail on arm64
Summary: devel/llvm38 and later fail on arm64
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: arm64 Any
: --- Affects Some People
Assignee: Brooks Davis
URL:
Keywords:
Depends on:
Blocks: 201763
  Show dependency tree / graph
 
Reported: 2016-09-01 16:52 UTC by Andrew Turner
Modified: 2016-09-02 15:53 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (brooks)

Attachments
Build log (767.01 KB, text/plain)
2016-09-01 16:52 UTC, Andrew Turner 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Turner freebsd_committer freebsd_triage 2016-09-01 16:52:38 UTC 
Created attachment 174293 [details]
Build log

llvm later than 3.8 fail to package on arm64 with failures similar to the following:
pkg-static: Unable to access file /wrkdirs/usr/ports/devel/llvm38/work/stage/usr/local/%%COMPILER_RT%%llvm38/asan_blacklist.txt: No such file or directory

I think we need to define COMPILER_RT to @comment in PLIST_SUB similar to how OPENMP is already done.

This is also an issue in devel/llvm39 and devel/llvm-devel.
Comment 1 commit-hook freebsd_committer freebsd_triage 2016-09-01 22:07:04 UTC 
A commit references this bug:

Author: brooks
Date: Thu Sep  1 22:06:32 UTC 2016
New revision: 421246
URL: https://svnweb.freebsd.org/changeset/ports/421246

Log:
  Attempt to fix build on platforms that don't support COMPILER_RT.

  Make GOLD an amd64-only option (not belived to be a functional change).

  PR:		212312

Changes:
  head/devel/llvm-devel/Makefile
  head/devel/llvm38/Makefile
  head/devel/llvm39/Makefile
Comment 2 Andrew Turner freebsd_committer freebsd_triage 2016-09-01 23:10:47 UTC 
I've started a test build to check if it's fixed.
Comment 3 Andrew Turner freebsd_committer freebsd_triage 2016-09-02 15:04:19 UTC 
devel/llvm-devel, devel/llvm38, and devel/llvm39 now build as expected.