Hi Markus,
On 24/10/16 22:01, Markus Geimer wrote:
Hi Kenneth,
On 22/10/16 14:29, Markus Geimer wrote:
Hi Kenneth,
I noticed that some easyconfigs (e.g., ncurses) specify
toolchainopts = {'optarch': True}
This makes me wonder about three things. First, I was under
the impression that 'optarch = True' was the default, i.e.,
isn't the above setting in the easyconfig redundant?
Yes, it is indeed redundant, the 'optarch' toolchain option is enabled
by default.
OK, so these are probably copy/pasted leftovers from times where
'optarch' was not yet the default, right?
Indeed, there's no point in adding this at all...
Maybe a clean sweep to remove this from all easyconfigs in the repo is
worthwhile here?
Maybe. But it "only" affects 1110 easyconfigs at the moment. Do you
want a single PR? ;-)
As long as the PR *only* gets rid of "'optarch': True", yes, sure, do it
in a single PR, since that changes doesn't require a full test report of
all easyconfigs involved...
Got it -- and it now makes sense. I believe what confused me most
was the fact that there are multiple options with the same name but
different semantics.
Yeah, I see where the confusion comes from...
But the 'optarch' toolchain option and --optarch configuration setting
are very related though, so it made sense to use the same name.
Maybe we should add a clarifying note that explains the difference
between both at
http://easybuild.readthedocs.io/en/latest/Controlling_compiler_optimization_flags.html
?
Adding some words about the toolchain option here would make sense,
IMHO. It also controls compiler optimization flags, right?
Indeed, I'll look into that.
Do you feel any enhancements need to be made to the documentation to
clarify things?
I couldn't find any description of the available toolchainopts in the
docs. Especially knowing about things like 'pic' or how to generically
enable C++11 support are valuable at times.
Well, there's the (auto-generated) page here:
http://easybuild.readthedocs.io/en/latest/version-specific/toolchain_opts.html
.
That's just rechewing the output of "eb --avail-toolchain-opts intel"
though...
Is that what you were looking for?
Basically, yes. It could be improved (e.g., by consolidating options
in a "generic" part and then only list additional toolchain-specific
ones, and maybe by adding a few more words or examples here and there),
but is a good starting point. However, when I was searching for
'toolchainopts', i.e., the easyconfig parameter, I didn't get to that
page. I also couldn't find other links to it. So you really need to
know the right keyword to search for it...
It's linked from a subsection of the landing page, see
http://easybuild.readthedocs.io/en/latest/#overview-of-version-specific-auto-generated-documentation-pages
.
But not finding it via 'toolchainopts' as a search query is clearly an
issue, I'll look into fixing that too.
I feel 'toolchainopts' at least deserves a small subsection in
http://easybuild.readthedocs.io/en/latest/Writing_easyconfig_files.html#common-easyconfig-parameters
.
And have some more extensive documentation on some of the commonly used
toolchain options (pic, usempi, openmp, optarch, opt, strict) is
probably worthwhile indeed.
regards,
Kenneth
Markus
--
Dr. Markus Geimer
Juelich Supercomputing Centre
Institute for Advanced Simulation
Forschungszentrum Juelich GmbH
52425 Juelich, Germany
Phone: +49-2461-61-1773
Fax: +49-2461-61-6656
E-Mail: [email protected]
WWW: http://www.fz-juelich.de/jsc
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzender des Aufsichtsrats: MinDir Dr. Karl Eugen Huthmacher
Geschaeftsfuehrung: Prof. Dr.-Ing. Wolfgang Marquardt (Vorsitzender),
Karsten Beneke (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
