Re: [Patch] Add 'default' to -foffload=; document that flag [PR67300]

[Sandra Loosemore <sandra@codesourcery.com>]

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