tree fe343f990ef43cc2085b01ca2a7a7581e405538e
parent 6ef1b6455821f0d1c6b61ba91ba0cadd374d4049
author David Benjamin <davidben@google.com> 1546136073 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1546472361 +0000

Fix CFI annotations in p256-x86_64-asm.pl.

This was caught by in-progress work to test unwind information. It was
incorrect at two instructions: immediately before we jump to
.Lpoint_double_shortcut$x. This is needed because
ecp_nistz256_point_add$x tries to be clever about not unwinding the
stack frame in its tail call.

It's also unlikely that the SEH handlers in this file are correct at
this point, but that will be handled separately while overhauling
everything else here. (For Win64, probably the only ABI-compliant option
is to just properly unwind the stack frame. Without a custom handler,
Win64 unwind codes are very restrictive.)

Bug: 181
Change-Id: I9f576d868850312d6c14d1386f8fbfa85021b347
Reviewed-on: https://boringssl-review.googlesource.com/c/33946
Commit-Queue: David Benjamin <davidben@google.com>
Reviewed-by: Adam Langley <agl@google.com>