All of lore.kernel.org
 help / color / mirror / Atom feed
From: Segher Boessenkool <segher@kernel.crashing.org>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: 440 ebony patch seems to have broken 85xx build on powerpc.git
Date: Fri, 25 May 2007 00:08:28 +0200	[thread overview]
Message-ID: <bd721e3985ba0b54ea1a00c2f4acd993@kernel.crashing.org> (raw)
In-Reply-To: <200705242316.29713.arnd@arndb.de>

> 1. Pass -Wa,-many to gcc
> This will really allow any possible instruction to be assembled, 
> including
> the old POWER architecture, but also includes all the other strange
> stuff like -maltivec, -mspe, -me500, ...

And it doesn't prevent the compiler from compiling insns
that can never work on the platform; for example, the
compiler in the original bug report will happily put isel
insns all over the place.

> 2. protect the use of special instructions with .machine directives.
> You can write all the inline assemblies like
>
> 	.machine push
> 	.machine 440
> 	<440 specific instruction>
> 	.machine pop
>
> This should work in any reasonably recent version of binutils, meaning
> that we don't need to pass stuff like -Wa,-m440 any more.

[You don't need the assembler flag at all, just the compiler
flag -mcpu=440].

This of course works, but is ultra-silly to do in a file
that will never run on anything other than 440 systems.


Segher

  reply	other threads:[~2007-05-24 22:08 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-24 17:13 440 ebony patch seems to have broken 85xx build on powerpc.git Dave Jiang
2007-05-24 17:37 ` Josh Boyer
2007-05-24 17:43   ` Dave Jiang
2007-05-24 18:02 ` Sergei Shtylyov
2007-05-24 18:09   ` Josh Boyer
2007-05-25  1:27     ` David Gibson
2007-05-24 19:05 ` Segher Boessenkool
2007-05-24 19:39   ` Josh Boyer
2007-05-24 20:18     ` Segher Boessenkool
2007-05-24 20:21     ` Arnd Bergmann
2007-05-24 20:24       ` Josh Boyer
2007-05-24 21:07         ` Mark A. Greer
2007-05-24 21:16         ` Arnd Bergmann
2007-05-24 22:08           ` Segher Boessenkool [this message]
2007-05-24 22:03         ` Segher Boessenkool
2007-05-24 22:02       ` Segher Boessenkool
2007-05-25  1:30         ` David Gibson
2007-05-25 14:01           ` Segher Boessenkool
2007-05-24 22:09   ` Segher Boessenkool

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bd721e3985ba0b54ea1a00c2f4acd993@kernel.crashing.org \
    --to=segher@kernel.crashing.org \
    --cc=arnd@arndb.de \
    --cc=linuxppc-dev@ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.