UO-OACISS / tau2

TAU Performance System Public Mirror (Updated every night at midnight, USA Pacific Time)
http://tau.uoregon.edu
Other
36 stars 15 forks source link

support new cray compiler names #7

Closed lukebroskop closed 1 year ago

lukebroskop commented 1 year ago

Hi, By default spack is trying to set crayCC, craycc, crayftn as tau build options for c++, cc, and fortran. crayCC, craycc, crayftn are "wrapper-less" approaches to using CCE. Is it possible to add crayCC, craycc, crayftn to configure script?

Thanks! Luke

khuck commented 1 year ago

Possible? yes - in the configure script, there are places where it checks if the OS is craycnl, and if so, assumes cc, CC and ftn are the compilers. Just look for those places...

lukebroskop commented 1 year ago

Oh, I'll create a PR. Thanks!

wspear commented 1 year ago

Note that this PR should fix the TAU package: https://github.com/spack/spack/pull/34175 However there are still advantages to supporting the direct cray compilers.

lukebroskop commented 1 year ago

Thanks @wspear, I had done a different fix to the package. But your fix is better!