[all-commits] [llvm/llvm-project] af54d1: [NVPTX] Set the atomic inling threshold when targe...

Joseph Huber via All-commits all-commits at lists.llvm.org
Thu Mar 23 14:30:26 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: af54d1e852850edcc7b9485851320d9ebf1be4fe
      https://github.com/llvm/llvm-project/commit/af54d1e852850edcc7b9485851320d9ebf1be4fe
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2023-03-23 (Thu, 23 Mar 2023)

  Changed paths:
    M clang/lib/Basic/Targets/NVPTX.cpp
    M clang/test/CodeGen/atomics-inlining.c

  Log Message:
  -----------
  [NVPTX] Set the atomic inling threshold when targeting NVPTX directly

Since Clang 16.0.0 users can target the `NVPTX` architecture directly
via `--target=nvptx64-nvidia-cuda`. However, this does not set the
atomic inlining size correctly. This leads to spurious warnings and
emission of runtime atomics that are never implemented. This patch
ensures that we set this to the appropriate pointer width. This will
always be 64 in the future as `nvptx64` will only be supported moving
forward.

Fixes: https://github.com/llvm/llvm-project/issues/61410

Reviewed By: tra

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




More information about the All-commits mailing list