This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: Change -malign-branch= to accept a comma separated list
- From: Jan Beulich <jbeulich at suse dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>, Fangrui Song <maskray at google dot com>
- Cc: Hongjiu Lu <hongjiu dot lu at intel dot com>, Shengchen Kan <shengchen dot kan at intel dot com>, Binutils <binutils at sourceware dot org>
- Date: Fri, 17 Jan 2020 10:05:36 +0100
- Subject: Re: [PATCH] x86: Change -malign-branch= to accept a comma separated list
- References: <20200117044822.mmi2hlqka2s67xwb@google.com> <CAMe9rOqFGRP-5eee5QEM_RuEEpBnqBLRiaxSKfyv5OAHz=Nr5Q@mail.gmail.com>
On 17.01.2020 06:02, H.J. Lu wrote:
> On Thu, Jan 16, 2020 at 8:49 PM Fangrui Song via binutils
> <binutils@sourceware.org> wrote:
>>
>> It is a convention to use a comma separated list.
>>
>> -fsanitize=address,undefined
>> -fsanitize=null,alignment
>> -fpatchable-function-entry=3,1
>> -finstrument-functions-exclude-function-list=a,b
>> -mrecip=all,!sqrt
>> -mfpmath=sse,387
>
> -malign-branch=TYPE[+TYPE...] (default: jcc+fused+jmp)
>
> is done on purpose since you can't pass comma in assembler options
> from GCC driver to assembler with -Wa,....
How about permitting both + and , ?
Jan