[all-commits] [llvm/llvm-project] 43999d: [spirv][amdgpu] Set atomic size in the clang targe...

Jon Chesterfield via All-commits all-commits at lists.llvm.org
Tue Feb 25 10:31:32 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 43999deb370113945ef86680014f838f55315ee7
      https://github.com/llvm/llvm-project/commit/43999deb370113945ef86680014f838f55315ee7
  Author: Jon Chesterfield <jonathanchesterfield at gmail.com>
  Date:   2025-02-25 (Tue, 25 Feb 2025)

  Changed paths:
    M clang/lib/Basic/Targets/SPIR.h

  Log Message:
  -----------
  [spirv][amdgpu] Set atomic size in the clang target info (#128569)

Problem identified by Joseph. The openmp device runtime uses
__scoped_atomic_load_n and similar which presently hit

```
error: large atomic operation may incur significant performance
      penalty; the access size (4 bytes) exceeds the max lock-free size (0 bytes) [-Werror,-Watomic-alignment]
```

This is because the spirv class doesn't set the corresponding field. The
base does, but only if there's a host toolchain, which there isn't.



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