Skip to content

Commit

Permalink
[NvlinkWrapper] Fix -pluing not consuming its argument
Browse files Browse the repository at this point in the history
Summary:
Sometimes `clang` will pass `-plugin` when doing LTO, which should be
correctly consumed by the nvlink wrapper. Right now it was leaving the
`plugin.so` argument as a regular input, which would cause it to error
on the `.so` input.
  • Loading branch information
jhuber6 committed Sep 22, 2024
1 parent 38666e6 commit 68e2b69
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion clang/tools/clang-nvlink-wrapper/NVLinkOpts.td
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ def library_S : Separate<["--", "-"], "library">, Flags<[HelpHidden]>,
def library_EQ : Joined<["--", "-"], "library=">, Flags<[HelpHidden]>,
Alias<library_path>;

def plugin : Joined<["--", "-"], "plugin">,
def plugin : JoinedOrSeparate<["--", "-"], "plugin">,
Flags<[HelpHidden, WrapperOnlyOption]>;

def arch : Separate<["--", "-"], "arch">,
Expand Down

0 comments on commit 68e2b69

Please sign in to comment.