Skip to content

Commit 9e41f82

Browse files
author
Erich Keane
authored
[SYCL] Fix DeclareSYCLBuiltins for round-trip setting. (#3491)
Upstream LLVMadded a round-trip functionality to the command line args so that we can generate a full set of command line arguments from our existing state. At the moment, this is 'assert' build only, but it seems the intent is to make this perminent for future functionality. We will have to analyze the full set of our command line args to make sure we do this right (that is, that it gets set right the 2nd time), but this one fixes the DeclareSYCLBuiltins to unblock the pulldown's build.
1 parent 9da3104 commit 9e41f82

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

clang/lib/Frontend/CompilerInvocation.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3496,6 +3496,9 @@ void CompilerInvocation::GenerateLangArgs(const LangOptions &Opts,
34963496
if (Opts.getSignReturnAddressKey() ==
34973497
LangOptions::SignReturnAddressKeyKind::BKey)
34983498
GenerateArg(Args, OPT_msign_return_address_key_EQ, "b_key", SA);
3499+
3500+
if (Opts.DeclareSPIRVBuiltins)
3501+
GenerateArg(Args, OPT_fdeclare_spirv_builtins, SA);
34993502
}
35003503

35013504
bool CompilerInvocation::ParseLangArgs(LangOptions &Opts, ArgList &Args,

0 commit comments

Comments
 (0)