[PATCH] D13449: [WinEH] Recognize CoreCLR personality function
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 5 15:31:32 PDT 2015
rnk added a comment.
In http://reviews.llvm.org/D13449#260132, @JosephTremoulet wrote:
> I'm not wedded to the name `isWinCFIPersonality` if people are opposed to conflating WinCFI with use of these handlers. I just thought MSVC didn't make sense with CLR included. I thought about `isWinEHPersonality` and variants, but since (a) we don't use these on e.g. MinGW and (b) CoreCLR will likely use these on non-Windows, tying it to WinCFI seemed most apt.
I agree, MSVC doesn't make sense. I don't think "CFI" makes sense, since that's mainly about call frame information, which x86 Windows doesn't even have. My first thought was that it should be `isWinEHPersonality`. I rationalized that the CoreCLR personality comes from Windows even if it works elsewhere, and `__gxx_personality_seh0` comes from Unix even though it runs on Windows.
David likes `isFuncletEHPersonality`, and I think that's a winner for me. Does that sound like a reasonable shade of chartreuse for the bikeshed? :)
http://reviews.llvm.org/D13449
More information about the llvm-commits
mailing list