tra added a comment. `-Xoffload-linker=<triple> <arg>`
The syntax is confusing. Normally only `triple` would be the argument for `-Xoffload-linker` option. Having both `-Xoffload-linker` and `-Xoffload-linker=` variants also looks odd to me. In effect you're making `-Xoffload-linker=foo` a full option (as opposed to it being an option `-Xoffload-linker= ` + argument `foo`) with a separate argument that follows. I guess that might work, but it's a rather unconventional use of command line parser, IMO. I think the main issue with this approach is that it makes the command line hard to understand. When one sees `-Xsomething=a -b` it's impossible to tell whether `-b` is a regular option or something to be passed to `-Xsomething=a`. My assumption would be the former as `-Xsomething=` already got its argument `a` and should have no business grabbing the next one. I think it would work better if the option could use a `-` or`_` for the variant that passes the triple. E.g. `-Xoffload-linker-nvptx64=-foo` or `-Xoffload-linker-nvptx64 -foo` would be easily interpretable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126226/new/ https://reviews.llvm.org/D126226 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
