From f72786adddfa4d98fe3d3ed7db22fb6d1636fd67 Mon Sep 17 00:00:00 2001 From: Artem Gindinson Date: Fri, 6 Mar 2020 16:15:14 +0300 Subject: [PATCH] [SYCL][NFC] Fix a misleading comment regarding the SYCL flow Signed-off-by: Artem Gindinson --- clang/lib/Driver/ToolChains/Clang.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index 611a2e1b9dd47..dbc94c79dfb97 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -3944,10 +3944,13 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, // Check number of inputs for sanity. We need at least one input. assert(Inputs.size() >= 1 && "Must have at least one input."); // CUDA/HIP compilation may have multiple inputs (source file + results of - // device-side compilations). OpenMP and SYCL device jobs also take the host - // IR as a second input. Module precompilation accepts a list of header files - // to include as part of the module. All other jobs are expected to have - // exactly one input. + // device-side compilations). + // OpenMP device jobs take the host IR as a second input. + // SYCL host jobs accept the integration header from the device-side + // compilation as a second input. + // Module precompilation accepts a list of header files to include as part + // of the module. + // All other jobs are expected to have exactly one input. bool IsCuda = JA.isOffloading(Action::OFK_Cuda); bool IsHIP = JA.isOffloading(Action::OFK_HIP); bool IsOpenMPDevice = JA.isDeviceOffloading(Action::OFK_OpenMP);