This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] x86: Pass -O0 to assembler for some tests

On Mon, Mar 18, 2019 at 7:37 PM Jan Beulich <JBeulich@suse.com> wrote:
>
> >>> On 18.03.19 at 02:21, <hjl.tools@gmail.com> wrote:
> >       * testsuite/gas/i386/att-regs.d: Pass -O0 to assembler.
> >       * testsuite/gas/i386/avx512bw-intel.d: Likewise.
> >       * testsuite/gas/i386/avx512bw.d: Likewise.
> >       * testsuite/gas/i386/avx512f-intel.d: Likewise.
> >       * testsuite/gas/i386/avx512f.d: Likewise.
> >       * testsuite/gas/i386/disp32.d: Likewise.
> >       * testsuite/gas/i386/intel-regs.d: Likewise.
> >       * testsuite/gas/i386/pseudos.d: Likewise.
> >       * testsuite/gas/i386/x86-64-disp32.d: Likewise.
> >       * testsuite/gas/i386/x86-64-pseudos.d: Likewise.
>
> It would be pretty nice if there was at least one sentence here as
> to why this change is needed and/or wanted. Repeating only what
> goes into the ChangeLog is not very useful.
>
> Fundamentally my question is: Isn't -O0 the default anyway? Why
> would it be necessary to specify this on the command line of any
> test case?
>

I have a patch to enable optimization by default via a configure
switch.

-- 
H.J.
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]