Skip to content

Commit c72b39c

Browse files
author
Artem Gindinson
authored
[SYCL][NFC] Fix a misleading comment regarding the SYCL flow (#1266)
Signed-off-by: Artem Gindinson <artem.gindinson@intel.com>
1 parent f0197ab commit c72b39c

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

0 commit comments

Comments
 (0)