[Bug other/67300] New: -foffload* undocumented

[Bug other/67300] New: -foffload* undocumented

[jsm28 at gcc dot gnu.org]

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

            Bug ID: 67300
           Summary: -foffload* undocumented
           Product: gcc
           Version: 6.0
            Status: UNCONFIRMED
          Keywords: openacc
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: jsm28 at gcc dot gnu.org
                CC: tschwinge at gcc dot gnu.org
  Target Milestone: ---

The -foffload= and -foffload-abi= options have no documentation in invoke.texi.

[Bug other/67300] -foffload* undocumented

[tschwinge at gcc dot gnu.org]

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

Thomas Schwinge <tschwinge at gcc dot gnu.org> 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 <tschwinge at gcc dot gnu.org> ---
(As you have found already), -foffload=[...] currently is only documented on
<https://gcc.gnu.org/wiki/Offloading>, and in the patch submission email,
<http://news.gmane.org/find-root.php?message_id=%3C20141011144900.GA16368%40msticlxl57.ims.intel.com%3E>.
 This documentation should be moved into the GCC users manual.

[Bug other/67300] -foffload* undocumented

[burnus at gcc dot gnu.org]

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

Tobias Burnus <burnus at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |burnus at gcc dot gnu.org

--- Comment #2 from Tobias Burnus <burnus at gcc dot gnu.org> ---
(In reply to Thomas Schwinge from comment #1)
> … in the patch submission email,
> <http://news.gmane.org/find-root.php?message_id=%3C20141011144900.
> 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

[burnus at gcc dot gnu.org]

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

--- Comment #3 from Tobias Burnus <burnus at gcc dot gnu.org> ---
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

[cvs-commit at gcc dot gnu.org]

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

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tobias Burnus <burnus@gcc.gnu.org>:

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

commit r12-1871-g33c4e466243f2d0459d9a4d0cadc1b38a3741f3e
Author: Tobias Burnus <tobias@codesourcery.com>
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]

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

--- Comment #5 from Tobias Burnus <burnus at gcc dot gnu.org> ---
Done: -foffload= and new -foffload-options= are now documented.

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

[Bug other/67300] -foffload* undocumented

[tschwinge at gcc dot gnu.org]

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

Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|81886                       |

--- Comment #6 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
(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