Skip to content

Commit 87c57b3

Browse files
committed
[SYCL][CUDA] include PTX in new driver
Somehow we still need ptx for sycl . Fix failures https://github.com/intel/llvm/actions/runs/13014198774/job/36299123697 ``` error: nvFatbinAddCubin(handle, file.image, file.size, arch, identName) failed with error fatbinary elf mismatch: elf size doesn't match user-specified size ```
1 parent 248b2d2 commit 87c57b3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

clang/lib/Driver/ToolChains/Cuda.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ void NVPTX::Assembler::ConstructJob(Compilation &C, const JobAction &JA,
517517
static bool shouldIncludePTX(const ArgList &Args, StringRef InputArch) {
518518
// The new driver does not include PTX by default to avoid overhead.
519519
bool includePTX = !Args.hasFlag(options::OPT_offload_new_driver,
520-
options::OPT_no_offload_new_driver, true);
520+
options::OPT_no_offload_new_driver, false); // INTEL
521521
for (Arg *A : Args.filtered(options::OPT_cuda_include_ptx_EQ,
522522
options::OPT_no_cuda_include_ptx_EQ)) {
523523
A->claim();

0 commit comments

Comments
 (0)