diff options
author | Andy Ayers <andya@microsoft.com> | 2019-06-21 08:32:37 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-06-21 08:32:37 -0700 |
commit | e31c78ba38273055ef0ca36de26f31ffabe521ac (patch) | |
tree | e21871bf81b51cf7e4d8131339beba789ecdd7b6 | |
parent | d1e7ef0f64b00fe2ab45bb0a893491528545de24 (diff) | |
download | coreclr-e31c78ba38273055ef0ca36de26f31ffabe521ac.tar.gz coreclr-e31c78ba38273055ef0ca36de26f31ffabe521ac.tar.bz2 coreclr-e31c78ba38273055ef0ca36de26f31ffabe521ac.zip |
JIT: update epilog exit size assert for x86 (#25302)
With the advent of #25251 we can now see the epilog exit sequence on x86 vary
by as much as 6 bytes. Update the assert accordingly.
Fixes #25301.
-rw-r--r-- | src/jit/emit.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/jit/emit.cpp b/src/jit/emit.cpp index 98fb188140..e7c582feb9 100644 --- a/src/jit/emit.cpp +++ b/src/jit/emit.cpp @@ -2059,9 +2059,10 @@ void emitter::emitEndFnEpilog() // underestimation of the epilog size is harmless (since the EIP // can not be between instructions). assert(emitEpilogCnt == 1 || - (emitExitSeqSize - newSize) <= 5 // delta between size of various forms of jmp (size is either 6 or 5) + (emitExitSeqSize - newSize) <= 6 // delta between size of various forms of jmp (size is either 6 or 5, + // or a 5 byte mov plus 2 byte jmp) // and various forms of ret (size is either 1 or 3). The combination can - // be anything been 1 and 5. + // be anything between 1 and 6. ); emitExitSeqSize = newSize; } |