On Sat, Jul 27, 2019 at 01:22:57PM -0500, Segher Boessenkool wrote:
> On Sat, Jul 27, 2019 at 02:56:23PM +0930, Alan Modra wrote:
> > The patch also introduces ASM_DEFAULT_EXTRA for the altivec variant
> > targets so as to enable -maltivec by default.
> 
> That is a behaviour change I think?  I don't know if it is correct...

This -maltivec is the one passed to the assembler, so no real gcc
behaviour change.

> It doesn't by default use -mabi=altivec as well, for example?
> 
> > Bootstrapped and regression tested powerpc64le-linux, and a number of
> > powerpc variants built to inspect flags passed to the assembler.  OK
> > for mainline?
> 
> Did you try with those *altivec targets as well?  Well I guess you did,
> or why else did you do this change?

Actually I spotted the need for a fix by grepping over the sources,
but I did build a powerpc-linuxaltivec target.

-- 
Alan Modra
Australia Development Lab, IBM

Reply via email to