New exception chaining implementation #409
Conversation
|
Thanks a lot for doing this! I haven't looked at the generated code in detail yet, but I couldn't find any major issues at first glance. To get proper coverage on the CI systems, just push your druntime/dmd-testsuite commits to the ldc-developers repositories. Oh, and there might be another dmd-testsuite test or two you can enable now, just grep the source for |
|
Feel free to merge after the build is green. |
| // the number of infos we had before the push | ||
| std::stack<size_t> nInfos; | ||
| // constructs the landing pad | ||
| void constructLandingPad(IRLandingPadScope scope); |
There was a problem hiding this comment.
The scope is changed in the function.
|
The build is green, but I am pretty sure that the pull request breaks win64 exception handling since |
|
The Win64 build is currently broken. Just merge the pull request - I try to fix the Win64 build then. |
New exception chaining implementation
Ideal staging branch v2.062-b1 for druntime
No description provided.