From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <binutils-return-38107-listarch-binutils=sources.redhat.com@sources.redhat.com>
Received: (qmail 15642 invoked by alias); 21 Feb 2005 20:05:15 -0000
Mailing-List: contact binutils-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:binutils-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/binutils/>
List-Post: <mailto:binutils@sources.redhat.com>
List-Help: <mailto:binutils-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: binutils-owner@sources.redhat.com
Received: (qmail 15494 invoked from network); 21 Feb 2005 20:05:08 -0000
Received: from unknown (HELO avtrex.com) (67.116.42.149)
  by sourceware.org with SMTP; 21 Feb 2005 20:05:08 -0000
Received: from [192.168.0.35] ([192.168.0.35] unverified) by avtrex.com with Microsoft SMTPSVC(5.0.2195.6713);
	 Mon, 21 Feb 2005 12:05:07 -0800
Message-ID: <421A3EEE.80707@avtrex.com>
Date: Mon, 21 Feb 2005 22:42:00 -0000
From: David Daney <ddaney@avtrex.com>
User-Agent: Mozilla Thunderbird 1.0 (X11/20041206)
MIME-Version: 1.0
To: Ian Lance Taylor <ian@wasabisystems.com>
CC:  binutils@sourceware.org
Subject: Re: News MIPS option -mno-shared
References: <20041209153403.5294.qmail@gossamer.airs.com>
In-Reply-To: <20041209153403.5294.qmail@gossamer.airs.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-OriginalArrivalTime: 21 Feb 2005 20:05:07.0256 (UTC) FILETIME=[A3AF6B80:01C51850]
X-SW-Source: 2005-02/txt/msg00479.txt.bz2

Ian Lance Taylor wrote:
> I'd like to see whether anybody has an opinion about this patch before
> I check it in.
> 
> This patch adds a new option to the MIPS assembler: -mno-shared.
> Normally the .cpload pseudo-op generates code which looks like this:
> 
> 	lui	$gp,%hi(_gp_disp)
> 	addiu	$gp,$gp,%lo(_gp_disp)
> 	addu	$gp,$gp,.cpload argument
> 
> With -mno-shared, the .cpload pseudo-op will generate code that looks
> like this:
> 
> 	lui	$gp,%hi(_gp)
> 	addiu	$gp,$gp,%lo(_gp)
.
.
.
> Does anybody want to comment on this patch before I check it in?

I like this, but just one belated nit...

as --help says nothing about the new option.

David Daney