[all-commits] [llvm/llvm-project] ab1ee9: [flang][cuda] Remove the need of special compile d...

[Valentin Clement (バレンタイン クレメン) via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ab1ee912be95a864827aadd33175bfeec026f000
      https://github.com/llvm/llvm-project/commit/ab1ee912be95a864827aadd33175bfeec026f000
  Author: Valentin Clement (バレンタイン クレメン) <clementval at gmail.com>
  Date:   2025-01-29 (Wed, 29 Jan 2025)

  Changed paths:
    M flang/CMakeLists.txt
    M flang/include/flang/Runtime/CUDA/init.h
    M flang/lib/Optimizer/Builder/Runtime/Main.cpp
    A flang/test/Lower/CUDA/cuda-init.cuf

  Log Message:
  -----------
  [flang][cuda] Remove the need of special compile definition for CUFInit (#124965)

This patch addresses post commit review comments from #124859. 

The extra compile definition is not necessary and goes against the
effort to separate the runtimes from the flang compiler itself. The
function declaration for `CUFInit` can be accessed anyway since the
header are always present. The insertion of the call is only based on
the language feature options from the folding context.

A program compiled with cuda enabled but no cufruntime would just fail
at link time as expected.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications