This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR62120
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Ilya Tocar <tocarip dot intel at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 1 Sep 2014 12:55:52 +0200
- Subject: Re: [PATCH] PR62120
- Authentication-results: sourceware.org; auth=none
- References: <20140901104314 dot GB26266 at msticlxl7 dot ims dot intel dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Sep 01, 2014 at 02:43:14PM +0400, Ilya Tocar wrote:
> Hi, this patch adds checks for registers availability, when
> alternative/numeric name is used.
> Bootstraps/passes make-check on x86-64.
> Ok for trunk?
>
> ChangeLog:
>
> gcc/
>
> 2014-09-01 Ilya Tocar <ilya.tocar@intel.com>
>
> * varasm.c (decode_reg_name_and_count): Check availability for
> registers from ADDITIONAL_REGISTER_NAMES.
Please mention the PR in the ChangeLog entry and add some testcases
(can be gcc.target/i386/, but we should have it tested).
Does this change anything on say register short sil __asm ("sil"); in 32-bit
mode (when it IMHO should be rejected too?)?
Jakub