This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] MIPS/GCC: Mark text contents as code or data
- From: Paul Hua <paul dot hua dot gm at gmail dot com>
- To: "Maciej W. Rozycki" <macro at imgtec dot com>
- Cc: Matthew Fortune <Matthew dot Fortune at imgtec dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Catherine Moore <clm at codesourcery dot com>, mfortune at gmail dot com
- Date: Wed, 14 Mar 2018 17:10:13 +0800
- Subject: Re: [PATCH] MIPS/GCC: Mark text contents as code or data
- Authentication-results: sourceware.org; auth=none
- References: <alpine.DEB.2.20.17.1611081807350.10580@tp.orcam.me.uk> <6D39441BF12EF246A7ABCE6654B0235380ACAB31@HHMAIL01.hh.imgtec.org> <alpine.DEB.2.20.17.1611161712230.10580@tp.orcam.me.uk>
I noticed that data-sym-pool.c fails on -O0 flags.
-O0 output :
-----------------cut----------------------
frob:
.frame $17,8,$31 # vars= 0, regs= 1/0, args= 0, gp= 0
.mask 0x00020000,0
.fmask 0x00000000,0
addiu $sp,-8
sd $17,0($sp)
move $17,$sp
lw $2,$L4
move $sp,$17
ld $17,0($sp)
addiu $sp,8
jr $31
.type __pool_frob_3, @object
__pool_frob_3:
.align 2
$L3:
.word __gnu_local_gp
$L4:
.word 305419896
.type __pend_frob_3, @function
__pend_frob_3:
.insn
.end frob
.size frob, .-frob
.ident "GCC: (gcc trunk r258495 mips64el o32 n32 n64) 8.0.1
20180313 (experimental)"
-------------end----------------------------------------------
Is it expected ? maybe we should add skip-if -O0 flags.
Paul Hua
On Thu, Nov 17, 2016 at 1:15 AM, Maciej W. Rozycki <macro@imgtec.com> wrote:
> On Tue, 15 Nov 2016, Matthew Fortune wrote:
>
>> I'm a little concerned the expected output tests may be fragile over
>> time but let's wait and see.
>
> Indeed, but I'd rather see false negatives than false positives or no
> coverage at all. And I hope the pieces of expected assembly quoted will
> help telling any false negatives and actual regressions apart very easily.
>
>> OK to commit.
>
> Applied now, thanks for your review.
>
> Maciej