dstuttard accepted this revision.
dstuttard added a comment.
This revision is now accepted and ready to land.
Herald added a reviewer: jdoerfert.
Herald added subscribers: jplehr, sstefan1.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155429/n
This revision was automatically updated to reflect the committed changes.
Closed by commit rG7940888c5987: [AMDGPU] Intrinsic to expose s_wait_event for
export ready (authored by dstuttard).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138216/new/
dstuttard updated this revision to Diff 477423.
dstuttard added a comment.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Updated - adding test for builtin
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138216/new/
https://re
dstuttard added a comment.
I think that the reason we don't see the same failure for e.g. OpenACC is that
there's a unittest/Frontend that requires OpenACC - if I comment out that
subdir in the unittests/CMakeLists.txt it fails llvm-config test in the same
way for OpenACC. Similarly, if I add F