Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for LLVM 16 #209

Open
athas opened this issue Nov 18, 2023 · 2 comments
Open

Support for LLVM 16 #209

athas opened this issue Nov 18, 2023 · 2 comments

Comments

@athas
Copy link

athas commented Nov 18, 2023

Oclgrind does not seem to support LLVM 16 yet. The first showstopper is that clang has changed the name of the include directory that contains opencl-c.h. From the release notes:

Clang’s resource directory path previously included the full clang version. It now includes only the major version. The new resource directory is $prefix/lib/clang/$CLANG_MAJOR_VERSION and can be queried using clang -print-resource-dir, just like before.

@jrprice
Copy link
Owner

jrprice commented Nov 20, 2023

LLVM 15 isn't yet supported either.

I have WIP changes to migrate to opaque pointers which is the main blocker on upgrading. They're almost done, so I will do my best to find some time in the near future to get that finished off so that we can support LLVM 15 and 16. Sorry for the delays!

@tkanarsky
Copy link

Any updates on this? I just tried to take a stab at porting it to LLVM 17 and quickly found myself over my head. In some cases it's pretty trivial to pull the pointee type info from e.g. the owning instr but in your business logic where we're hopping around and dereferencing things, there doesn't seem to be a good way other than keeping an out-of-band mapping from pair<address_space, Value*> -> pointee_type.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants