Bug 201599 - lang/clang-cheri should be defined as x86 only
Summary: lang/clang-cheri should be defined as x86 only
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: arm Any
: --- Affects Some People
Assignee: Brooks Davis
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-15 18:01 UTC by Sean Bruno
Modified: 2015-07-15 23:33 UTC (History)
1 user (show)

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

Attachments
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 Sean Bruno freebsd_committer freebsd_triage 2015-07-15 18:01:39 UTC 
I don't think there's any desire to have the CHERI version of clang built or supported for non-x86 architectures right now (maybe aarch64 in the future).

Can we add:

ONLY_FOR_ARCHS= amd64 i386
ONLY_FOR_ARCHS_REASON= Only supported on amd64 and i386
Comment 1 Brooks Davis freebsd_committer freebsd_triage 2015-07-15 22:51:39 UTC 
Hmm.  It's probably mostly useful on amd64 in practice, but it should work anywhere clang-devel does.  What is the motivation for this change?
Comment 2 Sean Bruno freebsd_committer freebsd_triage 2015-07-15 22:54:59 UTC 
(In reply to Brooks Davis from comment #1)
I get the following linker error with clang in an armv6 jail.  I didn't know if you guys wanted to support this port for other architectures.

http://chips.ysv.freebsd.org/data/11armv6-default/2015-07-11_19h23m27s/logs/errors/clang-cheri-3.7.d20150512.log
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-07-15 23:31:52 UTC 
A commit references this bug:

Author: brooks
Date: Wed Jul 15 23:31:40 UTC 2015
New revision: 392245
URL: https://svnweb.freebsd.org/changeset/ports/392245

Log:
  Mark broken on armv6 as it fails to link there and currently wouldn't be very
  useful.

  PR:		201599
  Submitted by:	sbruno

Changes:
  head/lang/clang-cheri/Makefile
Comment 4 Brooks Davis freebsd_committer freebsd_triage 2015-07-15 23:33:07 UTC 
We'll eventually want this working on ARM so we can support the SoCkit board as an integrated CHERI development platform, but for now I've marked it broken on armv6.