[Bug other/67300] -foffload* undocumented

[tschwinge at gcc dot gnu.org via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

Thomas Schwinge  changed:

   What|Removed |Added

 Depends on|81886   |

--- Comment #6 from Thomas Schwinge  ---
(In reply to Tobias Burnus from comment #5)
> Done: -foffload= and new -foffload-options= are now documented.

Thanks for your work on this (which I have not yet reviewed, having just
returned from vacations).  ;-)

Should we get this (after some time) into the GCC 11 branch, to be able to fix
up the libgomp testcases there, too?


> To do: Still undocumented:
>   -foffload-abi=[lp64|ilp32]

Not sure if we really want to properly document that one: that's an internal
thing, not to be touched by users.


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81886
[Bug 81886] Means to determine at runtime foffload targets specified at compile
time

[Bug other/67300] -foffload* undocumented

[burnus at gcc dot gnu.org via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

--- Comment #5 from Tobias Burnus  ---
Done: -foffload= and new -foffload-options= are now documented.

To do: Still undocumented:
  -foffload-abi=[lp64|ilp32]

[Bug other/67300] -foffload* undocumented

[cvs-commit at gcc dot gnu.org via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

--- Comment #4 from CVS Commits  ---
The master branch has been updated by Tobias Burnus :

https://gcc.gnu.org/g:33c4e466243f2d0459d9a4d0cadc1b38a3741f3e

commit r12-1871-g33c4e466243f2d0459d9a4d0cadc1b38a3741f3e
Author: Tobias Burnus 
Date:   Tue Jun 29 15:58:21 2021 +0200

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

As -foffload={options,targets,targets=options} is very convoluted,
it has been split into -foffload=targets (supporting the old syntax
for backward compatibilty) and -foffload-options={options,target=options}.

Only the new syntax is documented.

Additionally, -foffload=default is supported, which can reset the
devices after -foffload=disable / -foffload=targets to the default,
if needed.

gcc/ChangeLog:

PR other/67300
* common.opt (-foffload=): Update description.
(-foffload-options=): New.
* doc/invoke.texi (C Language Options): Document
-foffload and -foffload-options.
* gcc.c (check_offload_target_name): New, split off from
handle_foffload_option.
(check_foffload_target_names): New.
(handle_foffload_option): Handle -foffload=default.
(driver_handle_option): Update for -foffload-options.
* lto-opts.c (lto_write_options): Use -foffload-options
instead of -foffload.
* lto-wrapper.c (merge_and_complain, append_offload_options):
Likewise.
* opts.c (common_handle_option): Likewise.

libgomp/ChangeLog:

PR other/67300
* testsuite/libgomp.c-c++-common/reduction-16.c: Replace
-foffload=nvptx-none= by -foffload-options=nvptx-none= to
avoid disabling other offload targets.
* testsuite/libgomp.c-c++-common/reduction-5.c: Likewise.
* testsuite/libgomp.c-c++-common/reduction-6.c: Likewise.
* testsuite/libgomp.c/target-44.c: Likewise.

[Bug other/67300] -foffload* undocumented

[burnus at gcc dot gnu.org via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

--- Comment #3 from Tobias Burnus  ---
Patch to document -foffload=
  https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573081.html

Still undocumented:
  -foffload-abi=[lp64|ilp32]

[Bug other/67300] -foffload* undocumented

[burnus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

Tobias Burnus  changed:

   What|Removed |Added

 CC||burnus at gcc dot gnu.org

--- Comment #2 from Tobias Burnus  ---
(In reply to Thomas Schwinge from comment #1)
> … in the patch submission email,
>  GA16368%40msticlxl57.ims.intel.com%3E>.

Actually working link:
https://gcc.gnu.org/legacy-ml/gcc-patches/2014-10/msg01057.html

[Bug other/67300] -foffload* undocumented

[tschwinge at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67300

Thomas Schwinge  changed:

   What|Removed |Added

   Keywords||openmp
 Status|UNCONFIRMED |NEW
   Last reconfirmed||2015-09-21
 CC||bernds at gcc dot gnu.org,
   ||iverbin at gcc dot gnu.org,
   ||jakub at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Thomas Schwinge  ---
(As you have found already), -foffload=[...] currently is only documented on
, and in the patch submission email,
.
 This documentation should be moved into the GCC users manual.