Adds openmp offload for XSBench, RSBench, miniFE#4
Conversation
ggeorgakoudis
left a comment
There was a problem hiding this comment.
Please reply to comments and address issues
config.yaml
Outdated
| run: 'env OMP_NUM_THREADS=1 OMP_PROC_BIND=true ./XSBench' | ||
| input: '-t 1 -k 1 -s small' | ||
| measure: | ||
| input: '-t 1 -m event -s small' |
There was a problem hiding this comment.
Is this input compatible with the other configurations (omp, seq)?
config.yaml
Outdated
| run: 'env OMP_NUM_THREADS=1 OMP_PROC_BIND=true ./rsbench' | ||
| input: '-t 1 -s small' | ||
| measure: | ||
| input: '-s small -m event -t 1' |
There was a problem hiding this comment.
Is this input compatible with the other configurations (seq, omp)?
config.yaml
Outdated
| ], | ||
| omp-offload: [ 'cd openmp45/src; | ||
| make -j CC=clang CXX=clang++ | ||
| CFLAGS="-v -O3 -fopenmp -fopenmp-targets=nvptx64-nvidia-cuda |
| print_CLI_error(); | ||
| } | ||
| - else | ||
| + else if( strcmp(arg, "-t") == 0 ) |
There was a problem hiding this comment.
Is this to enable multi-threaded target creation? Is an option on the number of threads correct? Shouldn't be 1 for correct program execution?
| + } | ||
| // n_gridpoints (-g) | ||
| - if( strcmp(arg, "-g") == 0 ) | ||
| + else if( strcmp(arg, "-g") == 0 ) |
There was a problem hiding this comment.
Change to if, otherwise -t precludes -g
| char * arg = argv[i]; | ||
|
|
||
| + // This will be ignored | ||
| + if( strcmp(arg, "-t") == 0 ) |
There was a problem hiding this comment.
Is an option on the number of threads correct? Shouldn't be 1 for correct program execution?
config.yaml
Outdated
|
|
||
| XSBench: | ||
| fetch: 'git clone https://github.com/ANL-CESAR/XSBench.git' | ||
| fetch: 'git clone https://github.com/ANL-CESAR/XSBench.git; |
There was a problem hiding this comment.
Please add the command to clone/checkout the specific commit that this patch works with. Also for RSBench fetch. We should do it for every repo that needs a patch.
|
Possibly add: miniQMC |
- Move offload applications to another configuration file. - Add Clang and IBM compilation toolchain for applications. - Extends Harness to parse information out of the nvlink info - The parsed data are stored under the results/results-'application name'.yaml file - Add miniQMC application
d4d3ca2 to
17bcb73
Compare
This patch adds compiling configurations for XSBench, RSBench, and miniFE using the OpenMP offload programming model.