search

ecmwf-ifs / fiat

The Fortran IFS and Arpege Toolkit
Apache License 2.0
7 stars 19 forks source link

Add PAPI functionality to Dr Hook #18

Open ioanhadade opened 4 months ago

FussyDuck commented 4 months ago

CLA assistant check
All committers have signed the CLA.

wdeconinck commented 4 months ago

For sake of development, add C to the COMPONENTS in this line: https://github.com/ecmwf-ifs/fiat/pull/18/files#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR26

wdeconinck commented 4 months ago

@MartynF any occurrence of long_long should be replaced with long long. See https://github.com/icl-utk-edu/papi/blob/master/src/papi.h#L554-L559

MartynF commented 4 months ago

Did fiat-print-bindings require OpenMP in C? It looks like it has pragmas!

MartynF commented 4 months ago

@MartynF any occurrence of long_long should be replaced with long long. See https://github.com/icl-utk-edu/papi/blob/master/src/papi.h#L554-L559

In version 7.... which isn't distributed with majority use OS's including current HPC right now ....

wdeconinck commented 4 months ago

Did fiat-print-bindings require OpenMP in C? It looks like it has pragmas!

Yes it does, but this is a standalone executable that does not even link to fiat.

In version 7.... which isn't distributed with majority use OS's including current HPC right now ....

Right let's keep it then. Thanks for clarifying.