This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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 PR41879 (large mgrid regression)


Hi,

On Wed, 20 Jan 2010, Jakub Jelinek wrote:

> On Wed, Jan 20, 2010 at 01:46:19PM +0100, Michael Matz wrote:
> > It doesn't because I overwrite the options (which would have -msse2 on 
> > x86) with dg-options, and there's no way to just _add_ to the current 
> > options, except via implementing something like add_options_for_ieee (just 
> > for vectorizer options).  Pff, I think I'm going to checkin this patch, 
> > making the test active only on x86(-64), so that I can hardcode -msse2.
> 
> Why not have two dg-options lines instead, one with -msse2 and { target i?86-*-* x86_64-*-* },
> the other without both?

Looking at gfortran.dg/vect/vect.exp also other targets need special 
options, I don't want to replicate all this in 9 dg-options lines.


Ciao,
Michael.


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