This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: Initialize broadcast_op.bytes to 0
- From: "Jan Beulich" <JBeulich at suse dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: "Igor V Tsimbalist" <igor dot v dot tsimbalist at intel dot com>, <binutils at sourceware dot org>
- Date: Thu, 26 Jul 2018 09:57:38 -0600
- Subject: Re: [PATCH] x86: Initialize broadcast_op.bytes to 0
- References: <CAMe9rOpJ573hfc-3g1AqP5CQrayyR-993yhvqMqPDQbSALQLQQ@mail.gmail.com>
>>> On 26.07.18 at 17:51, <hjl.tools@gmail.com> wrote:
> On Thu, Jul 26, 2018 at 8:02 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>> Also, wouldn't you better clear ->bytes again in case the function later
>>> returns an error, leading to the next template to be looked at?
>>
>> Good point. I will take a look.
>>
>
> This is what I checked in.
Is that enough? This sets the value to zero once while parsing aiui,
but not between multiple attempts to match the parsing result
against templates.
Jan