This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RS6000] powerpc64 -mcmodel=medium large symbol offsets
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Alan Modra <amodra at gmail dot com>
- Date: Wed, 11 Sep 2013 16:56:14 -0400
- Subject: Re: [RS6000] powerpc64 -mcmodel=medium large symbol offsets
- Authentication-results: sourceware.org; auth=none
- References: <20130906071329 dot GB3430 at bubble dot grove dot modra dot org> <CAGWvny=9tx2QX7+DCvzNWt3S=AB6HWsxbT4rTkNbO8xSq-aArg at mail dot gmail dot com> <20130906233608 dot GA2643 at bubble dot grove dot modra dot org> <20130907024438 dot GB2643 at bubble dot grove dot modra dot org> <20130909090703 dot GC2643 at bubble dot grove dot modra dot org> <20130911111249 dot GE2643 at bubble dot grove dot modra dot org>
On Wed, Sep 11, 2013 at 7:12 AM, Alan Modra <amodra@gmail.com> wrote:
> On Mon, Sep 09, 2013 at 06:37:03PM +0930, Alan Modra wrote:
>> gcc/
>> * config/rs6000/predicates.md (add_cint_operand): New.
>> (reg_or_add_cint_operand): Use add_cint_operand.
>> * config/rs6000/rs6000.md (largetoc_high_plus): Restrict offset
>> using add_cint_operand.
>> (largetoc_high_plus_aix, small_toc_ref): Likewise.
>> gcc/testsuite/
>> * gcc.target/powerpc/medium_offset.c: New.
>
> I missed seeing one testcase regression caused by this patch. :-(
> gcc.c-torture/compile/pr41634.c at -O3 gets an "insn does not satisfy
> its constraints". Fixed with the following. OK to apply?
>
> * config/rs6000/rs6000.c (toc_relative_expr_p): Use add_cint_operand.
Okay.
Thanks, David