[all-commits] [llvm/llvm-project] 4d3c01: [CUDA] Do not embed a fatbinary when using the new...

Joseph Huber via All-commits all-commits at lists.llvm.org
Thu Jun 23 12:41:07 PDT 2022

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4d3c010f1d012ade0ca36fdd7bc8b8baeb8df1f2
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2022-06-23 (Thu, 23 Jun 2022)

  Changed paths:
    M clang/lib/Driver/Driver.cpp
    M clang/lib/Driver/ToolChains/Cuda.cpp
    M clang/test/Driver/cuda-openmp-driver.cu
    M clang/test/Driver/cuda-phases.cu

  Log Message:
  [CUDA] Do not embed a fatbinary when using the new driver

Previously, when using the new driver we created a fatbinary with the
PTX and Cubin output. This was mainly done in an attempt to create some
backwards compatibility with the existing CUDA support that embeds the
fatbinary in each TU. This will most likely be more work than necessary
to actually implement. The linker wrapper cannot do anything with these
embedded PTX files because we do not know how to link them, and if we
did want to include multiple files it should go through the
`clang-offload-packager` instead. Also this didn't repsect the setting
that disables embedding PTX (although it wasn't used anyway).

Reviewed By: tra

Differential Revision: https://reviews.llvm.org/D128441

More information about the All-commits mailing list