[clang] [libc] [Clang] Implement resource directory headers for common GPU intrinsics (PR #110179)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 5 11:07:41 PST 2024


================
@@ -0,0 +1,155 @@
+//===-- amdgpuintrin.h - AMDPGU intrinsic functions -----------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef __AMDGPUINTRIN_H
+#define __AMDGPUINTRIN_H
+
+#ifndef __AMDGPU__
+#error "This file is intended for AMDGPU targets or offloading to AMDGPU"
+#endif
+
+#include <stdbool.h>
+#include <stdint.h>
----------------
AaronBallman wrote:

I think it's a bit surprising that including this header will transitively define `bool` in C modes; I would expect we'd use `_Bool` in this header file.

As for C89 modes, I think this will still work okay, but you should have explicit test coverage for C89.

https://github.com/llvm/llvm-project/pull/110179


More information about the cfe-commits mailing list