[PATCH] D12533: [WinEH] Add llvm.eh.padparam intrinsic

[Reid Kleckner via llvm-commits]

rnk added a comment.

In http://reviews.llvm.org/D12533#237415, @JosephTremoulet wrote:

> I'm certainly not wedded to keeping padparam this general.  The specific signature that I need is `exception_aggr addrspace(1)* @llvm.eh.intrinsicname()` (or I could get by with `i8 addrspace(1)* @llvm.eh.intrinsicname()` and bitcasts).  I guess for my specific case `intrinsicname` would be `get_clr_exception` or something.  If you'd prefer that, let me know what you think the name should be and whether you want any of the ones you listed included in this patch, and I'll be happy to change it.


I guess the addressspace(1) thing has to be baked into the signature for you, so we can't get by with `i8* @llvm.eh.exceptionpointer(token)` and `addresspacecast`. How about making it llvm_any_ty and using `i8 addressspace(1) * @llvm.eh.exceptionpointer.p1i8(token)`?


http://reviews.llvm.org/D12533