Hi Alan, Thank you for doing this!
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... 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? In general it isn't great that we have these target configs that no one ever tests, or *can* test properly. Ugh. The patch is okay for trunk. Thanks! Segher
