Bug 215516 - lang/spidermonkey170: use clang on armv6
Summary: lang/spidermonkey170: use clang on armv6
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: arm Any
: --- Affects Many People
Assignee: Koop Mast
URL:
Keywords:
Depends on:
Blocks: 215517
  Show dependency treegraph
 
Reported: 2016-12-23 15:16 UTC by mikael.urankar
Modified: 2017-05-09 18:05 UTC (History)
1 user (show)

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


Attachments
use clang (2.71 KB, patch)
2016-12-23 15:16 UTC, mikael.urankar
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mikael.urankar 2016-12-23 15:16:25 UTC
Created attachment 178225 [details]
use clang

Hi,

clang can build sm170 with the attached patch.
The patch is similar to the one commited in ports r423893

build log (17MB)
http://mikael.urankar.free.fr/FreeBSD/arm/build_logs/spidermonkey170-17.0.0_4.log

Thanks in advance
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2017-02-03 13:05:14 UTC
spidermonkey170 builds on armv6 as of 20170122.  Is this PR still relevant?
Comment 2 mikael.urankar 2017-02-03 13:25:46 UTC
Yes the pr is still relevant. sm170 uses gcc49 as of today and it causes all sorts of problem on armv6.
Comment 3 commit-hook freebsd_committer 2017-05-09 18:05:07 UTC
A commit references this bug:

Author: kwm
Date: Tue May  9 18:04:59 UTC 2017
New revision: 440509
URL: https://svnweb.freebsd.org/changeset/ports/440509

Log:
  Use clang on armv6.

  PR:		215516
  Submitted by:	mikael.urankar@gmail.com

Changes:
  head/lang/spidermonkey170/Makefile
  head/lang/spidermonkey170/files/patch-assembler_wtf_Platform.h
  head/lang/spidermonkey170/files/patch-vm_NumericConversions.h