[PATCH] x86: Warn .insn instruction with length > 15 bytes
H.J. Lu
hjl.tools@gmail.com
Tue Feb 6 15:34:17 GMT 2024
On Tue, Feb 6, 2024 at 7:04 AM Michael Matz <matz@suse.de> wrote:
>
> > > > > I disagree. It was a warning until you changed it (without me really
> > > > > agreeing), and some hypothetical vendor could come and lift the limit,
> > > > > on precisely the basis that there are legal encodings exceeding it. I'm
> > > > > okay if this can be forced to be an error, but I continue to think that
> > > > > it ought to be a warning only by default.
> > > > >
> > > >
> > > > It is an error on both Intel and AMD processors. There is no
> > > > valid reason not to be an error at the moment.
> > >
> > > Jan gave you one. I would prefer for the assembler to not be anally
> >
> > That is not a valid reason.
>
> Yes it is.
>
> > There is no such processor in the foreseeable future.
>
> Doesn't matter.
>
When a warning is given, a decodable instruction should still
be generated. Assembler shouldn't generate something which
can't be decoded by default.
--
H.J.
More information about the Binutils
mailing list