aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Frontend/CompilerInvocation.cpp
diff options
context:
space:
mode:
authorHeejin Ahn <aheejin@gmail.com>2021-02-15 07:32:51 -0800
committerHeejin Ahn <aheejin@gmail.com>2021-02-17 12:14:11 -0800
commitda01a9db8bb4d0ba362e1956ef7fa9bdd5e3b464 (patch)
tree6a0101ef47844fca9dc1d8db344a606ce315ac01 /clang/lib/Frontend/CompilerInvocation.cpp
parent3a030c2f2fe3de62c5eba4f916ce51e5d1214a42 (diff)
downloadllvm-da01a9db8bb4d0ba362e1956ef7fa9bdd5e3b464.zip
llvm-da01a9db8bb4d0ba362e1956ef7fa9bdd5e3b464.tar.gz
llvm-da01a9db8bb4d0ba362e1956ef7fa9bdd5e3b464.tar.bz2
[WebAssemblly] Fix EHPadStack update in fixCallUnwindMismatches
Updating `EHPadStack` with respect to `TRY` and `CATCH` instructions have to be done after checking all other conditions, not before. Because we did this before checking other conditions, when we encounter `TRY` and we want to record the current mismatching range, we already have popped up the entry from `EHPadStack`, which we need to access to record the range. The `baz` call in the added test needs try-delegate because the previous TRY marker placement for `quux` was placed before `baz`, because `baz`'s return value was stackified in RegStackify. If this wasn't stackified this try-delegate is not strictly necessary, but at the moment it is not easy to identify cases like this. I plan to transfer `nounwind` attributes from the LLVM IR to prevent cases like this. The call in the test does not have `unwind` attribute in order to test this bug, but in many cases of this pattern the previous call has `nounwind` attribute. Reviewed By: tlively Differential Revision: https://reviews.llvm.org/D96711
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions