This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [bfin] Improve legitimize_pic_address
- From: Bernd Schmidt <bernds_cb1 at t-online dot de>
- To: Jie Zhang <jzhang918 at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 27 Nov 2007 13:11:55 +0100
- Subject: Re: [bfin] Improve legitimize_pic_address
- References: <46F246E0.2040905@gmail.com>
Jie Zhang wrote:
> legitimize_pic_address tried to legitimize
>
> (const:SI (plus:SI (symbol_ref:SI ("_ZNSs4_Rep20_S_empty_rep_storageE"))
> (const_int 12)))
>
> It put the SYMBOL_REF in (reg:SI 9 P1). But it failed to legitmize
> (const_int 12), since reload_in_progress == 1 and it could not force it
> in a register.
>
> Since Blackfin has instructions like
>
> DPREGS += IMM7
>
> we need not force such immediates into a register. This patch does that.
That seems alright, but what stops us from creating such a CONST with a
larger offset?
Bernd
--
This footer brought to you by insane German lawmakers.
Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen
Sitz der Gesellschaft Muenchen, Registergericht Muenchen HRB 40368
Geschaeftsfuehrer Thomas Wessel, William A. Martin, Margaret Seif