On Thu, 12 Oct 2017, Tsimbalist, Igor V wrote:
>       * unwind.inc (_Unwind_RaiseException_Phase2): Use FRAMES_P_DECL,
>       FRAMES_VAR_DECL_1, FRAMES_VAR_INC and FRAMES_P_UPDATE.
>       (_Unwind_RaiseException): Use FRAMES_VAR_DECL, FRAMES_VAR_P and
>       FRAMES_VAR.
>       (_Unwind_ForcedUnwind_Phase2): Use FRAMES_P_DECL,
>       FRAMES_VAR_DECL_1, FRAMES_VAR_INC, FRAMES_P_UPDATE.
>       (_Unwind_ForcedUnwind): Use FRAMES_VAR_DECL, FRAMES_VAR_P and
>       FRAMES_VAR.
>       (_Unwind_Resume): Use FRAMES_VAR_DECL, FRAMES_VAR_P and
>       FRAMES_VAR.
>       (_Unwind_Resume_or_Rethrow): Use FRAMES_VAR_DECL, FRAMES_VAR_P
>       and FRAMES_VAR.

I think you'll have to test this on a non-CET target too,
because it looks like this won't fly:

-  uw_install_context (&this_context, &cur_context);
+  uw_install_context (&this_context, &cur_context, FRAMES_VAR);

You'll be introducing a naked comma for the default empty
FRAMES_VAR.  I like the basic approach though, FWIW.

brgds, H-P

Reply via email to