Change default of VECTORISE_INLINE to "no" in Vectors

Comments (4)

1. 

   Erik Schnetter

   • removed comment

   We should ensure that this change of the default behaviour does not impact existing configurations. In many cases, existing configuration were hand-tuned for performance, or were designed according to similar configurations, and this work should not be undone by changing a default.

   In other words: This patch is okay to apply if corresponding VECTORISE_INLINE=yes statements are added to those configurations which don't set this option at the moment, in particular for those configurations that do not use the Intel compiler. (If a configuration uses the Intel compiler, then it is a judgement call whether it should be changed without testing.)

   On another note, machines not supported by Simfactory would probably not use vectorisation unless the user explicitly sets this, probably by copying from another configuration.

   • 2011-10-21T08:02:22+00:00
2. 

   Ian Hinder reporter

   • removed comment

   I changed all optionlists which used the default to explicitly specify VECTORISE_INLINE=yes in r1488 on 19-Oct:

   Set VECTORISE_INLINE = yes for all optionlists that do not set it already

   This corresponds to the current default in LSUThorns/Vectors. This default is about to be changed, and setting the variable in the optionlists means that changing the default should have no effect for machines in SimFactory.

   Hence, as far as simfactory is concerned, the change in the default of this variable is a null operation.

   • 2011-10-21T08:38:01+00:00
3. 

   Ian Hinder reporter

   • changed status to resolved
   • removed comment

   Committed in r33.

   • 2011-10-22T10:36:11+00:00
4. 

   Roland Haas

   • edited description
   • changed status to closed

   • 2019-02-21T20:23:08+00:00
5. Log in to comment