Update FAQ.md #2195
Update FAQ.md #2195
Conversation
FAQ.md
Outdated
|
|
||
| With a device PLATFORM (e.g., `nvidiaX` or `amdgpuX`), the compiler will generate OpenMP code for device offloading. | ||
|
|
||
| When using OpenMP offloading, it is however recommended to stick to the corresponding vendor compiler, so `ARCH=amdclang` for AMD, `ARCH={icc,icx,intel}` for Intel, and `ARCH=nvc` for NVidia. |
There was a problem hiding this comment.
Maybe note that icx is the default when using intel?
and ICC gonna be deprecated?
FAQ.md
Outdated
|
|
||
| Requires: `PLATFORM=nvidiaX` and `ARCH=cuda`. | ||
|
|
||
| Also requires DevitoPRO. |
There was a problem hiding this comment.
Available only with a DevitoPRO license ?
... or drop "also"
FAQ.md
Outdated
|
|
||
| Requires: `PLATFORM=amdgpuX` and `ARCH=hip`. | ||
|
|
||
| Also requires DevitoPRO. |
| Also requires DevitoPRO. | ||
|
|
||
|
|
||
| [top](#Frequently-Asked-Questions) |
There was a problem hiding this comment.
Since this is here, you should drop line 303
There was a problem hiding this comment.
I don't know what line 303 pointed to
Codecov Report
@@ Coverage Diff @@
## master #2195 +/- ##
=======================================
Coverage 87.06% 87.06%
=======================================
Files 228 228
Lines 40299 40299
Branches 7357 7357
=======================================
Hits 35088 35088
Misses 4625 4625
Partials 586 586 |
FAQ.md
Outdated
|
|
||
| #### LANGUAGE=openacc | ||
|
|
||
| Recommended: `PLATFORM=nvidiaX` and `ARCH=nvc`. |
There was a problem hiding this comment.
PLATFORM=nvidiaX is required here not recommended
|
GTG no? |
| @@ -244,26 +246,6 @@ These environment variables can either be set from the shell or programmatically | |||
| | programmatically | configuration['language'] = 'openmp' | | |||
|
|
|||
|
|
|||
| ### Table of values for Devito environment variables | |||
There was a problem hiding this comment.
removed as obsolete and unmaintanable
replaced suggesting the user to execute print_defaults instead ( a few lines above)
|
Now patched as per suggestions. Plus a few other tweaks. Merging |
No description provided.