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: Fix ld testcases on c6x


On Wed, May 11, 2011 at 01:10:23PM +0200, Bernd Schmidt wrote:
> +if { ![ld_assemble $as "$as_opt $srcdir/$subdir/tbss.s" tmpdir/tbss.o ]
> +     || ![ld_assemble $as "$as_opt $srcdir/$subdir/frame.s" tmpdir/frame.o] }

Passing options like this in the "source" parameter of ld_assemble
isn't the cleanest way.  Better would be to twiddle ASFLAGS (See
ld-frv/tls.exp for an example).  I'll OK your patch changed to use
ASFLAGS.

Best would be to add an "asflags" parameter to ld_assemble but I won't
ask for that since I probably wouldn't bother to do so myself..
Preapproved if you do care to go that route.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]