This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 5/6] Add Visium support to gas
- From: Alan Modra <amodra at gmail dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: binutils at sourceware dot org
- Date: Sat, 6 Dec 2014 13:33:57 +1030
- Subject: Re: [PATCH 5/6] Add Visium support to gas
- Authentication-results: sourceware.org; auth=none
- References: <cover dot 1417646850 dot git dot ebotcazou at gcc dot gnu dot org> <fee07d8c32de19e9586f02a00a656e3628d5f68f dot 1417646850 dot git dot ebotcazou at gcc dot gnu dot org> <20141205022551 dot GC20383 at bubble dot grove dot modra dot org> <3471125 dot 2qDhuGFaY0 at polaris>
On Fri, Dec 05, 2014 at 01:27:38PM +0100, Eric Botcazou wrote:
> > Also, I don't see any check on fx_subsy. You should emit an error if
> > fx_subsy is present and not handled.
>
> Neither tc-i386.c nor tc-sparc.c does that though.
Hmm. Looks like TC_VALIDATE_FIX_SUB will cause a complaint, so this
isn't necessary. This patch is OK, so I think the whole series can be
committed.
--
Alan Modra
Australia Development Lab, IBM