Skip to content

Commit f72786a

Browse files
author
Artem Gindinson
committed
[SYCL][NFC] Fix a misleading comment regarding the SYCL flow
Signed-off-by: Artem Gindinson <artem.gindinson@intel.com>
1 parent fd54d92 commit f72786a

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3944,10 +3944,13 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
39443944
// Check number of inputs for sanity. We need at least one input.
39453945
assert(Inputs.size() >= 1 && "Must have at least one input.");
39463946
// CUDA/HIP compilation may have multiple inputs (source file + results of
3947-
// device-side compilations). OpenMP and SYCL device jobs also take the host
3948-
// IR as a second input. Module precompilation accepts a list of header files
3949-
// to include as part of the module. All other jobs are expected to have
3950-
// exactly one input.
3947+
// device-side compilations).
3948+
// OpenMP device jobs take the host IR as a second input.
3949+
// SYCL host jobs accept the integration header from the device-side
3950+
// compilation as a second input.
3951+
// Module precompilation accepts a list of header files to include as part
3952+
// of the module.
3953+
// All other jobs are expected to have exactly one input.
39513954
bool IsCuda = JA.isOffloading(Action::OFK_Cuda);
39523955
bool IsHIP = JA.isOffloading(Action::OFK_HIP);
39533956
bool IsOpenMPDevice = JA.isDeviceOffloading(Action::OFK_OpenMP);

0 commit comments

Comments
 (0)