Bug#840186: [Debichem-devel] Bug#840186: Bug#840186: nwchem builds on amd64 with -march=native

[Apra, Edoardo]

Yes,

You are right. I mistakenly sent the -R version.

The new attachment should be the right one.

Could you please update the NWChem Debian source?

We already got a few report of failures from NWChem users.

Thanks, Edo


On 11/18/2016 03:16 AM, Michael Banck wrote:
> Hi,
>
>> On Fri, Nov 18, 2016 at 01:18:23AM +, Apra, Edoardo wrote:
>>> -  FOPTIONS +=  -mtune=native
>>> +  FOPTIONS += -march=native -mtune=native
> By the way, it seems this patch is reversed, as it adds -march=native.
>
>
> Michael
>

--- nwchem-6.6/src/config/makefile.h.orig	2016-11-17 22:26:29.497599135 +
+++ nwchem-6.6/src/config/makefile.h	2016-11-17 22:29:09.794581240 +
@@ -1956,7 +1956,7 @@
   FOPTIONS +=  -ff2c -fno-second-underscore
 endif
 ifeq ($(GNU_GE_4_6),true) 
-  FOPTIONS += -march=native -mtune=native
+  FOPTIONS +=  -mtune=native
   FOPTIONS += -finline-functions
 else
 ifeq ($(_GOT3DNOW),Y) 

Bug#840186: [Debichem-devel] Bug#840186: Bug#840186: nwchem builds on amd64 with -march=native

[Michael Banck]

Hi,

> On Fri, Nov 18, 2016 at 01:18:23AM +, Apra, Edoardo wrote:
> > -  FOPTIONS +=  -mtune=native
> > +  FOPTIONS += -march=native -mtune=native

By the way, it seems this patch is reversed, as it adds -march=native.


Michael

Bug#840186: [Debichem-devel] Bug#840186: nwchem builds on amd64 with -march=native

[Michael Banck]

tags 840186 +patch
tags 840186 +pending
thanks

On Fri, Nov 18, 2016 at 01:18:23AM +, Apra, Edoardo wrote:
> The attached patch disables the gfortran flag -march=native.
> Same patch can be found at the URL
> http://www.nwchem-sw.org/images/Disable_march_native.patch.gz

> --- nwchem-6.6/src/config/makefile.h  2016-11-17 22:29:09.794581240 +
> +++ nwchem-6.6/src/config/makefile.h.orig 2016-11-17 22:26:29.497599135 
> +
> @@ -1956,7 +1956,7 @@
>FOPTIONS +=  -ff2c -fno-second-underscore
>  endif
>  ifeq ($(GNU_GE_4_6),true) 
> -  FOPTIONS +=  -mtune=native
> +  FOPTIONS += -march=native -mtune=native
>FOPTIONS += -finline-functions
>  else
>  ifeq ($(_GOT3DNOW),Y) 

Thanks Edoardo, I've applied it to our packaging repository and will
look at uploading a new package with this patch later today.


Michael

Bug#840186: nwchem builds on amd64 with -march=native

[Apra, Edoardo]

The attached patch disables the gfortran flag -march=native.
Same patch can be found at the URL
http://www.nwchem-sw.org/images/Disable_march_native.patch.gz
--- nwchem-6.6/src/config/makefile.h	2016-11-17 22:29:09.794581240 +
+++ nwchem-6.6/src/config/makefile.h.orig	2016-11-17 22:26:29.497599135 +
@@ -1956,7 +1956,7 @@
   FOPTIONS +=  -ff2c -fno-second-underscore
 endif
 ifeq ($(GNU_GE_4_6),true) 
-  FOPTIONS +=  -mtune=native
+  FOPTIONS += -march=native -mtune=native
   FOPTIONS += -finline-functions
 else
 ifeq ($(_GOT3DNOW),Y) 

Bug#840186: nwchem builds on amd64 with -march=native

[Adrian Bunk]

Source: nwchem
Version: 6.6+r27746-2
Severity: grave

src/config/makefile{,-legacy}.h sets -march=native
and -mtune=native in amd64 builds.

-march=native means the code will only run on machines with a CPU
compatible with the one on the machine where the package
was built.

Building on a different buildd machine can also change
where the code can run - including breaking existing
setups in stable updates.

-mtune=native is less fatal, but in addition to (rare) problems
due to the fact that the program is compiled differently depending
on the build machine it makes the build non-reproducible.

Please remove -march=native and -mtune=native