[PATCH] D15405: [EHPersonality] Add a new personality enum to represent langindPad of token type

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 13:37:22 PST 2015


rnk added a comment.

In http://reviews.llvm.org/D15405#307850, @sanjoy wrote:

> Does it make sense to see a token returning personality as "wrapping" another existing personality?  I mean something like (psuedo code):


Joseph, what did you think was wrong with checking for token type? I think the original approach is simpler. It's a little funky when you have to materialize `resume {i8*, i32}` from separate values, but oh well.

I can imagine wanting to use some kind of relocating GC scheme in a function that uses C++ EH, and needing to name out the right __gxx_personality* symbol for the platform.


http://reviews.llvm.org/D15405





More information about the llvm-commits mailing list