#639: Change default of VECTORISE_INLINE to "no" in Vectors ------------------------------------+--------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: new Priority: major | Milestone: ET_2011_10 Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+---------------------------------------
Comment (by eschnett):
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.