exception fixes #162

pca006132 · 2022-01-14T19:33:05+08:00

pca006132 commented

2022-01-14 19:33:05 +08:00

Faster exception unwinding. (as we now only do a single phase unwinding)
Better backtrace. (as we now have forced unwinding)
Exception allocation. (names are not yet allocated)
Nested exception support. (error reporting is not yet done)

The first commit (use forced unwinding) is backward compatible with the old compiler, and does not do exception allocation and nested exception support.

The second commit should be used together with artiq exception branch, which contains the compiler modification required for this to work.

- Faster exception unwinding. (as we now only do a single phase unwinding) - Better backtrace. (as we now have forced unwinding) - Exception allocation. (names are not yet allocated) - Nested exception support. (error reporting is not yet done) The first commit (use forced unwinding) is backward compatible with the old compiler, and does not do exception allocation and nested exception support. The second commit should be used together with artiq exception branch, which contains the compiler modification required for this to work.

pca006132 added 1 commit 2022-01-14 19:33:06 +08:00

runtime/eh_artiq: use forced unwind 8923feceac

This patches ports the LLVM libunwind newly added forced unwinding
function. This enables us to run forced unwinding to obtain correct
backtrace when uncaught exceptions occur.

This patch also changes the exception handling scheme from the standard
two-phase unwinding to single phase using forced unwinding. This brings
some performance improvement and prepared for later nested exception
support. For nested exceptions, we will have to record the backtrace
regardless if the exception is an uncaught exception, as there can be
another exception being thrown while executing the finally block for
caught exceptions, and we will lose the backtrace if we don't store it
earlier before running the cleanup pads.

pca006132 force-pushed exception from b2171ff091 to 86f10e0ab5

2022-01-14 19:39:08 +08:00

Compare

sb10q reviewed 2022-01-14 21:47:59 +08:00

src/runtime/src/kernel/api.rs Outdated

						
				@ -205,3 +205,3 @@

				        api!(__artiq_personality = eh_artiq::artiq_personality),

				        api!(__artiq_raise = eh_artiq::raise),

				        api!(__artiq_reraise = eh_artiq::reraise),

				        // api!(__artiq_reraise = eh_artiq::reraise),