[PATCH] D77920: [Windows EH] Fix the order of Nested try-catches in $tryMap$ table

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 14:19:38 PDT 2020


rnk added a comment.

Sounds good, and the fix looks simple, but please add a test.

I'm surprised this change doesn't break an existing test, I'd expect us to have a test for the LSDA of a try-in-catch:

  try { }
  catch () {
    try { }
    catch () { }
  }

As I understand it, the outer TBME must appear before the inner one, and the current behavior puts them in the other way around, innermost first.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77920/new/

https://reviews.llvm.org/D77920





More information about the llvm-commits mailing list