[all-commits] [llvm/llvm-project] d70254: [mlir][nvvm] Add missing include to llvm-config.h....

Ingo Müller via All-commits all-commits at lists.llvm.org
Wed Mar 6 01:13:25 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d70254a623d7c23c493cc793756a5a9d758b0a14
      https://github.com/llvm/llvm-project/commit/d70254a623d7c23c493cc793756a5a9d758b0a14
  Author: Ingo Müller <ingomueller at google.com>
  Date:   2024-03-06 (Wed, 06 Mar 2024)

  Changed paths:
    M mlir/lib/Target/LLVM/NVVM/Target.cpp
    M utils/bazel/llvm-project-overlay/mlir/BUILD.bazel

  Log Message:
  -----------
  [mlir][nvvm] Add missing include to llvm-config.h. (#83998)

This is another follow-up of #83004. `NVVM/Target.cpp` uses the macro
`MLIR_NVPTXCOMPILER_ENABLED`, which is defined in `llvm-config.h` but
did not include that file, yielding a warning when compiled with
`-Wundef`. This PR adds the include.

~~This is another follow-up of #83004, which made the same change for
`MLIR_CUDA_CONVERSIONS_ENABLED`. As the previous PR, this PR commit
exposes mentioned CMake variable through `mlir-config.h` and uses the
macro that is introduced with the same name. This replaces the macro
`MLIR_NVPTXCOMPILER_ENABLED`, which the CMake files previously defined
manually.~~



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


More information about the All-commits mailing list