public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Nick Clifton <nickc@redhat.com>
To: JBeulich@suse.com
Cc: binutils@sourceware.org
Subject: Re: [PATCH] gas: require an operand to .startof.()/.sizeof.()
Date: Wed, 22 Feb 2017 16:31:00 -0000	[thread overview]
Message-ID: <87o9xu2nmq.fsf@redhat.com> (raw)

Hi Jan,

> gas/
> 2017-02-21  Jan Beulich  <jbeulich@suse.com>
> 
> 	* expr.c (operand): Handle missing operand to .startof.() and
> 	.sizeof.().
> 	* testsuite/gas/all/err-sizeof.s: New.

The cris architecture is not the only one affected by this new test.
The following are also affected:

 alpha-dec-vms 
 cr16-elf 
 crx-elf 
 d10v-elf 
 d30v-elf 
 epiphany-elf 
 fr30-elf 
 frv-elf 
 h8300-elf 
 h8300-rtems 
 i386-darwin 
 ip2k-elf 
 iq2000-elf 
 lm32-rtems4.10 
 m32c-elf 
 m32r-elf 
 m68hc11-elf 
 mep-elf 
 microblaze-elf 
 mn10200-elf 
 mn10300-elf 
 ms1-elf 
 msp430-elf 
 mt-elf 
 or1k-elf 
 pdp11-dec-aout 
 riscv32-elf 
 riscv64-elf 
 rl78-elf 
 rs6000-aix4.3.3 
 rx-elf 
 s390-linux 
 s390x-ibm-tpf 
 spu-elf 
 tilegx-linux-gnu 
 tilegxbe-linux-gnu 
 tilepro-elf 
 v850-elf 
 vax-netbsdelf 
 xc16x-elf 
 xgate-elf 
 xstormy16-elf 

Given the large number of targets affected, maybe the test itself needs
to be reconsidered ?  Or maybe split into two tests, one general one,
and one for targets which do support the difference of two symbols in an
expression ?

Cheers
  Nick
  

             reply	other threads:[~2017-02-22 16:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-22 16:31 Nick Clifton [this message]
2017-02-22 17:12 ` Jan Beulich
2017-02-23 10:20 ` Jan Beulich
  -- strict thread matches above, loose matches on Subject: below --
2017-02-21 10:15 Jan Beulich
2017-02-22  6:48 ` Alan Modra
2017-02-22 13:23 ` Hans-Peter Nilsson

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=87o9xu2nmq.fsf@redhat.com \
    --to=nickc@redhat.com \
    --cc=JBeulich@suse.com \
    --cc=binutils@sourceware.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).