This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [hsa merge 08/10] HSAIL BRIG description header file
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Ian Lance Taylor <ian at airs dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, Jeff Law <law at redhat dot com>
- Date: Sat, 16 Jan 2016 12:43:07 +0100
- Subject: Re: [hsa merge 08/10] HSAIL BRIG description header file
- Authentication-results: sourceware.org; auth=none
- References: <20160113173925 dot 220029649 at virgil dot suse dot cz> <20160113173925 dot 842609640 at virgil dot suse dot cz> <20160114130447 dot GF3017 at tucnak dot redhat dot com> <m31t9jtzb3 dot fsf at pepe dot airs dot com> <20160115101433 dot GJ3982 at virgil dot suse dot cz> <20160115103732 dot GS3017 at tucnak dot redhat dot com> <20160115120335 dot GU3017 at tucnak dot redhat dot com> <20160115172305 dot GP3982 at virgil dot suse dot cz>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Jan 15, 2016 at 06:23:05PM +0100, Martin Jambor wrote:
> BRIG_KIND_OPERAND_REGISTER = 0x300a,
> BRIG_KIND_OPERAND_STRING = 0x300b,
> BRIG_KIND_OPERAND_WAVESIZE = 0x3009c,
> BRIG_KIND_OPERAND_END = 0x300d
The above looks weird, I'd have expected BRIG_KIND_OPERAND_WAVESIZE
to be 0x300c instead. Bug in the standard?
As typedef uint16_t BrigKind16_t;, I'm afraid this doesn't even fit
into the data type. Note the original brig header you've posted
had this fixed.
Jakub