NFC: Region annotation intrnsics addition based on RFC from llvmpar

[xtian]

Summary

This is the 1st NFC patch for Region Annotation instrinsic addition

[neboat]

llvm/include/llvm/IR/Intrinsics.td

965

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.

[xtian]

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

llvm/include/llvm/IR/Intrinsics.td

965

Ok, I will make a change, i1 or int?

[xtian]

I would suggest we go with llvm_i32_ty.

[lebedev.ri]

Where's the latest RFC for this?

llvm/include/llvm/IR/Intrinsics.td

958–959

There is no RegionAnnotation.rst, not in trunk, not in this patch, not in parent differentials in stack,

[xtian]

Roman, TB is working .rst file.

RFC: Initial version

https://lists.llvm.org/pipermail/llvm-dev/2017-January/108906.html

RFC: Updated version based on feedback

http://lists.llvm.org/pipermail/llvm-dev/2018-October/126971.html

llvm/include/llvm/IR/Intrinsics.td

def int_experimental_gc_statepoint : Intrinsic<[llvm_token_ty],
                                 [Throws, ImmArg<0>, ImmArg<1>, ImmArg<3>, ImmArg<4>]>;
 
 def int_experimental_gc_result   : Intrinsic<[llvm_any_ty], [llvm_token_ty],
                                              [IntrReadMem]>;
 def int_experimental_gc_relocate : Intrinsic<[llvm_any_ty],
                                 [llvm_token_ty, llvm_i32_ty, llvm_i32_ty],
                                 [IntrReadMem, ImmArg<1>, ImmArg<2>]>;
 
 //===------------------------ Region Annotation Intrinsics ---------------===//
 // These are documented in docs/RegionAnnotation.rst
 
 // Region Annotation and Marker Intrinsics
 
 def int_region_entry : Intrinsic<[llvm_token_ty], [], []>;
 def int_region_exit : Intrinsic<[], [llvm_token_ty], []>;
-def int_region_marker : Intrinsic<[], [], []>;
+def int_region_marker : Intrinsic<[llvm_i1_ty], [], []>;
 
 //===------------------------ Coroutine Intrinsics ---------------===//
 // These are documented in docs/Coroutines.rst
 
 // Coroutine Structure Intrinsics.
 
 def int_coro_id : Intrinsic<[llvm_token_ty], [llvm_i32_ty, llvm_ptr_ty,
                              llvm_ptr_ty, llvm_ptr_ty],