libgomp/plugin/plugin-gcn.c: Use -foffload-options= in err msg
Commit Message
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
Comments
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
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.
@@ -2352,7 +2352,7 @@ isa_matches_agent (struct agent_info *agent, Elf64_Ehdr *image)
snprintf (msg, sizeof msg,
"GCN code object ISA '%s' does not match GPU ISA '%s'.\n"
- "Try to recompile with '-foffload=-march=%s'.\n",
+ "Try to recompile with '-foffload-options=-march=%s'.\n",
isa_s, agent_isa_s, agent_isa_gcc_s);
hsa_error (msg, HSA_STATUS_ERROR);