[PATCH] D64956: NFC: Region annotation intrnsics addition based on RFC from llvmpar

Xinmin Tian via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 14:33:36 PDT 2019


xtian marked an inline comment as done.
xtian added a comment.

I will the change,  do you want to make it i8, or int ?



================
Comment at: llvm/include/llvm/IR/Intrinsics.td:965
+def int_region_exit : Intrinsic<[], [llvm_token_ty], []>;
+def int_region_marker : Intrinsic<[], [], []>;
+
----------------
neboat wrote:
> In the LLVMPar RFC, we proposed that `int_region_marker` would return an `i1`. We needed that facility to use this intrinsic to model certain types of parallel control flow, e.g., Tapir.
Ok, I will make a change, i1 or  int? 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64956





More information about the llvm-commits mailing list