fhahn added a comment.

Another thing I noticed that there's verifier support missing. I think we 
should at least check that only a single `clang.arc.rv` bundle is specified 
(https://github.com/llvm/llvm-project/blob/main/llvm/lib/IR/Verifier.cpp#L3191).
 We should probably also enforce that the bundle is only provided for functions 
with an `i8*` return type. That can also be done after the main patch lands.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92808/new/

https://reviews.llvm.org/D92808

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to