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: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Jan Beulich <jbeulich at suse dot com>
- Cc: Fangrui Song <maskray at google dot com>, Shengchen Kan <shengchen dot kan at intel dot com>, Binutils <binutils at sourceware dot org>
- Date: Fri, 17 Jan 2020 08:23:26 -0800
- 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> <d687dd4c-6bbf-2eee-4bf9-ca33a9ed9440@suse.com>
On Fri, Jan 17, 2020 at 1:05 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> 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 , ?
>
I don't think it is necessary.
--
H.J.