Details
Diff Detail
Event Timeline
I realize that the wrapper may be removed in few months so let me know if this is irrelevant.
This change will make the flang script support nvfortran a bit better, but that's at the cost of some extra complexity. And this script has never been meant to be particularly scalable or extensible :) @ygribov , do you envisage more such options? Or is this a one off?
flang/tools/f18/flang | ||
---|---|---|
182 | -acc is not accepted by Flang, so it's in a yet different category. Also, the current comment is out-of-date: -fopenmp and -fopenacc _are_ in the same category as -I and -J. Would you mind updating it accordingly? | |
261–265 | This new case makes sense, but could you make it clear that this is specifically for nvfortan? Updating the comment would be fine. | |
264 |
I expect it to be one off because other relevant flags seem to already be supported. nvfortran's OpenACC is better than gfortran's one so it's generally preferred for OpenACC testing.
-acc is not accepted by Flang, so it's in a yet different category. Also, the current comment is out-of-date: -fopenmp and -fopenacc _are_ in the same category as -I and -J. Would you mind updating it accordingly?