Bug 215948 - head -r311147 (e.g.) for powerpc/powerpc64: clang does not support -Wa,-many and apparently does not need it.
Summary: head -r311147 (e.g.) for powerpc/powerpc64: clang does not support -Wa,-many ...
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: CURRENT
Hardware: powerpc Any
: --- Affects Only Me
Assignee: Justin Hibbits
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-01-10 20:16 UTC by Mark Millard
Modified: 2017-04-01 19:32 UTC (History)
1 user (show)

See Also:

Attachments
Avoid using -Wa,-many for clang for powerpc family (523 bytes, patch)
2017-01-10 20:21 UTC, Mark Millard 		no flags Details | Diff
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 Mark Millard 2017-01-10 20:16:40 UTC 
clang 3.9.1 does not support -Wa,-many :

# clang -c -Wa,-many locore64_simplified.S
clang: error: unsupported argument '-many' to option 'Wa,'

So -many should be avoided where it is not needed for
clang.
Comment 1 Mark Millard 2017-01-10 20:21:13 UTC 
Created attachment 178727 [details]
Avoid using -Wa,-many for clang for powerpc family

An alternate would be use of:

CFLAGS.gcc+= -msoft-float -Wa,-many
CFLAGS.clang+= -msoft-float

without the conditional logic, presuming such
is supported for the context.
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-01-22 06:00:51 UTC 
A commit references this bug:

Author: jhibbits
Date: Sun Jan 22 06:00:06 UTC 2017
New revision: 312614
URL: https://svnweb.freebsd.org/changeset/base/312614

Log:
  Don't pass -Wa,-many through clang, the integrated as doesn't support it.

  Our base binutils sets -many by default anyway, but external gcc may not do
  this.

  PR:	kern/215948
  Submitted by:	Mark Millard <markmi AT dsl-only DOT net>
  Reported by:	Mark Millard
  MFC after:	2 weeks

Changes:
  head/sys/conf/Makefile.powerpc
Comment 3 Justin Hibbits freebsd_committer freebsd_triage 2017-04-01 19:32:51 UTC 
MFC'd to stable/11 r316369