On 6/17/21 10:03 AM, Tobias Burnus wrote:
Updated version – only lightly tested. I think it is
consistent like that and the documentation should now be
comprehensive. (I will have to do some additional testing.)
Further comments and thoughts?
Hmmm, I had started to put together some comments on
grammar/punctuation/markup on the first version before the second
iteration showed up in my mailbox, but more critically I could not
figure out whether -foffload=default is supposed to be exactly identical
to the default behavior; if it isn't, it should be, or -foffload=default
ought to be renamed. So let's get that sorted out first. I suggest
reorganizing the documentation to first have a paragraph discussing the
default behavior, and then move on to how to modify it, with separate
paragraphs for enabling offload targets explicitly and on adding options
for offload compilation on all/some targets.
I think it would useful to add an example with a target list and
multiple options since I think the syntax looks pretty hairy.
-Sandra
