[COMMITTED] gas: change meaning of ; in the BPF assembler

Jan Beulich jbeulich@suse.com
Tue Nov 28 16:23:52 GMT 2023


On 28.11.2023 16:27, Jose E. Marchesi wrote:
> 
>> On 28.11.2023 15:05, Jose E. Marchesi wrote:
>>> The BPF assembler in clang uses semi-colon (;) to separate statements,
>>> not to be begin line comments.  This patch adapts the GNU assembler
>>> accordingly.
>>>
>>> Testsuite and documentation updated accordingly.
>>
>> Isn't this a behavioral change which also warrants a NEWS entry?
> 
> What about this:

Reads okay to me, thanks.

Jan

> diff --git a/gas/NEWS b/gas/NEWS
> index 643f0e6700e..6bfab08c5b8 100644
> --- a/gas/NEWS
> +++ b/gas/NEWS
> @@ -37,6 +37,10 @@
>  * Add support for various T-Head extensions (XTheadVector, XTheadZvlsseg
>    and XTheadZvamo) from version 2.3.0 of the T-Head ISA manual.
>  
> +* The BPF assembler now uses semi-colon (;) to separate statements, and
> +  therefore they cannot longer be used to begin line comments. This matches the
> +  behavior of the clang/LLVM BPF assembler.
> +
>  Changes in 2.41:
>  
>  * Add support for the KVX instruction set.



More information about the Binutils mailing list