rnk added a comment.

Looks functionally correct



================
Comment at: test/Sema/warn-unreachable-ms.c:42
+  }
+}
----------------
Can we add a test to exercise that this builds the right CFG?
```
__try {
  __try {
    f();
  } __except(1) {
    __leave; // should exit outer try
  }
  __leave;
  f(); // expected-warning{{never be executed}}
} __except(1) {
}
```


https://reviews.llvm.org/D36914



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to