On Wed, May 04, 2022 at 06:16:14PM +0200, Tobias Burnus wrote:
> See also https://gcc.gnu.org/gcc-12/changes.html#languages and
> https://gcc.gnu.org/onlinedocs/gcc/C-Dialect-Options.html#index-foffload
>
> -foffload= was never officially documented, albeit most users will
> have encountered it. Since GCC 12 it is - but the -foffload=-<flag>
> part is officially only handled by -foffload-options=, even if it
> works as legacy feature with -foffload= as well.
>
> OK for GCC 13?
>
> Tobias
>
> PS: Note that -foffload=amdgcn-amdhsa=-march=gfx908 and
> -foffload-options=amdgcn-amdhsa=-march=gfx908 are not identical.
> The former (legacy feature) will disable all other supported targets,
> such as nvptx, while the latter keeps the default set - and just passes
> that additional flag to the amdgcn target compiler.
> -----------------
> Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634
> München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas
> Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht
> München, HRB 106955
> libgomp/plugin/plugin-gcn.c: Use -foffload-options= in err msg
>
> While -foffload=-<flag> works (never documented legacy feature),
> the documented way is to use -foffload-options=.
>
> libgomp/ChangeLog:
>
> * plugin/plugin-gcn.c (isa_matches_agent): Suggest -foffload-options.
LGTM.
Jakub
