NFC: Fix typo in Statepoint documentation

[kavon]

Summary

The gc.statepoint intrinsic expects two i32's after the function argument, not i64's.

[kavon]

I'll note that the requirement that the # call args and flags are i32s conflicts with the rest of the documentation, namely, that the flags argument is 64 bits wide.

Is the documentation about the flags argument being an i64 also wrong, or is the intrinsic type checker wrong?

[sanjoy]

In D29112#655809, @kavon wrote:

I'll note that the requirement that the # call args and flags are i32s conflicts with the rest of the documentation, namely, that the flags argument is 64 bits wide.

Is the documentation about the flags argument being an i64 also wrong, or is the intrinsic type checker wrong?

Well, at least one of them is wrong. :) I'd go with changing the doc to avoid breaking out of tree users (since, clearly, they could not have been using a 64 bit flag "according to spec").

[Jim]

Do you need someone to commit this change for you?

[xgupta]

This gc.statepoint intrinsic reference is now documented in langref.rst in case it needs to be committed.

docs/Statepoints.rst

 Syntax:
 """""""
 
 ::
 
       declare token
         @llvm.experimental.gc.statepoint(i64 <id>, i32 <num patch bytes>,
                        func_type <target>, 
-                       i64 <#call args>, i64 <flags>,
+                       i32 <#call args>, i32 <flags>,
                        ... (call parameters),
                        i64 <# transition args>, ... (transition parameters),
                        i64 <# deopt args>, ... (deopt parameters),
                        ... (gc parameters))
 
 Overview:
 """""""""