tree d5311106ffd0cc017d181d0d9f1fc3d7d38d9bf5
parent a306b1b908cacf49d17cf77437d38da98ca5edcf
author David Benjamin <davidben@google.com> 1546138735 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1546472854 +0000

Fix beeu_mod_inverse_vartime CFI annotations and preamble.

This was also caught by the in-progress unwind tester. There are two
issues here.  First, .cfi_endproc must come after ret to fully cover the
function.  More importantly, this function is confused about whether it
has a frame pointer or not.

It looks like it does (movq %rsp, %rbp), and annotates accordingly, but
it does not actually use the frame pointer. It cannot. $y4 is rbp and
gets clobbered immediately after the preamble!

Remove this instruction and align the CFI annotations with a
frame-pointer-less function.

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