https://github.com/JonChesterfield updated
https://github.com/llvm/llvm-project/pull/131164
>From 092024bbf31b0677e6efbb0e6fc0cee7606ecb08 Mon Sep 17 00:00:00 2001
From: Jon Chesterfield
Date: Tue, 18 Mar 2025 15:57:02 +
Subject: [PATCH] [Headers] Implement spirvamdgcnintrin.h
---
clang/l
JonChesterfield wrote:
OK, we can't have spirv64-unknown-unknown at this point. But we could get this
compiling using a mixture of spirv intrinsics (where they exist) and amdgpu
intrinsics for spirv64-amd-amdhsa, leaving a preprocessor error for other
targets. I'll take a stab at that.
https:
https://github.com/JonChesterfield updated
https://github.com/llvm/llvm-project/pull/131164
>From d91671fdbb2aa9204f728747009459373bfd6553 Mon Sep 17 00:00:00 2001
From: Jon Chesterfield
Date: Thu, 13 Mar 2025 15:44:52 +
Subject: [PATCH 1/2] [Headers] Create stub spirintrin.h
---
clang/li
https://github.com/JonChesterfield updated
https://github.com/llvm/llvm-project/pull/131164
>From d91671fdbb2aa9204f728747009459373bfd6553 Mon Sep 17 00:00:00 2001
From: Jon Chesterfield
Date: Thu, 13 Mar 2025 15:44:52 +
Subject: [PATCH 1/2] [Headers] Create stub spirintrin.h
---
clang/li
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff c9d7f707c101896294d2f4ccbf26329894295dc9
1dbc0b25e69879a8799a03027e257ea68e1d596f --e
JonChesterfield wrote:
Well it's not pretty, but spirv64-amd-amdhsa sets both __AMDGPU__ and
__SPIRV64__ macros. Added a commit with an example that dispatches to amdgpu
intrinsics on the happy path and preprocessor error otherwise.
If you let that get to the spirv backend it falls over with `
JonChesterfield wrote:
The utility is having a place to fill things in incrementally as we get them
working, and thus use libc to drive the implementation of enough of spirv to
get things working. If you decline to have any spirv code until everything is
working I'll have to do the testing som
jhuber6 wrote:
We should probably wait until we have builtins, since there's no utility in
just having stubs right now.
https://github.com/llvm/llvm-project/pull/131164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
JonChesterfield wrote:
@jhuber6 I think we should have this despite the rejected
https://github.com/llvm/llvm-project/pull/131190.
Maybe we'll get some clang builtins for spirv. Otherwise some things can be
done with the asm label hack. Some can just be left as nop, e.g. a suspend that
does n
https://github.com/sarnex approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/131164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
JonChesterfield wrote:
Effectively a subset of https://github.com/llvm/llvm-project/pull/131190/, I'd
still like to land this and rebase 131190 on the grounds of signal to noise
ratio.
https://github.com/llvm/llvm-project/pull/131164
___
cfe-commits
https://github.com/JonChesterfield updated
https://github.com/llvm/llvm-project/pull/131164
>From d91671fdbb2aa9204f728747009459373bfd6553 Mon Sep 17 00:00:00 2001
From: Jon Chesterfield
Date: Thu, 13 Mar 2025 15:44:52 +
Subject: [PATCH] [Headers] Create stub spirintrin.h
---
clang/lib/He
https://github.com/JonChesterfield edited
https://github.com/llvm/llvm-project/pull/131164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
13 matches
Mail list logo
