[PATCH] X86: Update gas/NEWS for Intel APX.

[PATCH] X86: Update gas/NEWS for Intel APX.

[Cui, Lili]

gas/ChangeLog:

        * NEWS: Update gas/NEWS for Intel APX.
---
 gas/NEWS | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/gas/NEWS b/gas/NEWS
index be14ec66867..86a53fc59ea 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,5 +1,11 @@
 -*- text -*-
 
+* Support CFCMOV feature in Intel APX.
+
+* Support CCMP and CTEST feature in Intel APX.
+
+* Support zero-upper feature in Intel APX.
+
 * Add a .base64 directive to the assembler which allows base64 encoded
   binary data to be provided as strings.
   
-- 
2.34.1

Thanks,
Lili.

Re: [PATCH] X86: Update gas/NEWS for Intel APX.

[Jan Beulich]

On 11.07.2024 08:49, Cui, Lili wrote:
> --- a/gas/NEWS
> +++ b/gas/NEWS
> @@ -1,5 +1,11 @@
>  -*- text -*-
>  
> +* Support CFCMOV feature in Intel APX.
> +
> +* Support CCMP and CTEST feature in Intel APX.
> +
> +* Support zero-upper feature in Intel APX.
> +
>  * Add a .base64 directive to the assembler which allows base64 encoded
>    binary data to be provided as strings.
>    

Is there anything left for APX? I thought it's all done now (minus bugs
of course), in which case simply saying "full APX support" or something
along those lines may be better.

Jan

RE: [PATCH] X86: Update gas/NEWS for Intel APX.

[Cui, Lili]

> On 11.07.2024 08:49, Cui, Lili wrote:
> > --- a/gas/NEWS
> > +++ b/gas/NEWS
> > @@ -1,5 +1,11 @@
> >  -*- text -*-
> >
> > +* Support CFCMOV feature in Intel APX.
> > +
> > +* Support CCMP and CTEST feature in Intel APX.
> > +
> > +* Support zero-upper feature in Intel APX.
> > +
> >  * Add a .base64 directive to the assembler which allows base64 encoded
> >    binary data to be provided as strings.
> >
> 
> Is there anything left for APX? I thought it's all done now (minus bugs of
> course), in which case simply saying "full APX support" or something along
> those lines may be better.
> 

All instructions in the currently published APX spec are supported, agreed with you, there are still some minor bugs, just found one this morning.
In the disassembler, we print the old registers(ah,bh,ch and dh) for SETcc. I will fix it later. 

I thought about adding a summary like "full APX support", but I'm not sure if this is appropriate. I'll check it out and if it's ok, I'll add it as you suggest.

Thanks,
Lili.

Re: [PATCH] X86: Update gas/NEWS for Intel APX.

[Jan Beulich]

On 11.07.2024 12:35, Cui, Lili wrote:
>> On 11.07.2024 08:49, Cui, Lili wrote:
>>> --- a/gas/NEWS
>>> +++ b/gas/NEWS
>>> @@ -1,5 +1,11 @@
>>>  -*- text -*-
>>>
>>> +* Support CFCMOV feature in Intel APX.
>>> +
>>> +* Support CCMP and CTEST feature in Intel APX.
>>> +
>>> +* Support zero-upper feature in Intel APX.
>>> +
>>>  * Add a .base64 directive to the assembler which allows base64 encoded
>>>    binary data to be provided as strings.
>>>
>>
>> Is there anything left for APX? I thought it's all done now (minus bugs of
>> course), in which case simply saying "full APX support" or something along
>> those lines may be better.
> 
> All instructions in the currently published APX spec are supported, agreed with you, there are still some minor bugs, just found one this morning.
> In the disassembler, we print the old registers(ah,bh,ch and dh) for SETcc. I will fix it later. 

Right, as said - minus bugs.

> I thought about adding a summary like "full APX support", but I'm not sure if this is appropriate. I'll check it out and if it's ok, I'll add it as you suggest.

Thanks. And if done like that: Okay to put in.

Jan