[PATCH] Mangling for intrinsic names w/function type parameters
atrick at apple.com
Thu Oct 9 14:02:06 PDT 2014
This is my current thinking on this patch:
We do want to allow defining a single intrinsic in tablegen that can support any return type. Having a single intrinsic opcode will cleanup the code.
The only issue with the approach taken in this patch is that it mangles the entire signature of the target function into the patchpoint symbol. This seems unnecessary to me. I would prefer to see simple declarations for the intrinsics and use a bitcast when needed to pass the target function as i8*. I'm not aware of any use for the complex manglings, but we can debate that in the larger statepoint infrastructure review.
I believe Juergen has a patch that takes a simpler approach and I've asked him to post it.
More information about the llvm-commits