Skip to content

Commit

Permalink
Fix assert about maximum fragment split size (#92027)
Browse files Browse the repository at this point in the history
We want to support arbitrary `DOTNET_JitSplitFunctionSize` values
for stressing the unwind fragment splitting logic, but there are
limitations to the smallest size that can be split. Thus, we can
exceed a specified minimum. Loosen an assert that the last
fragment be less than the specified split size, since it might
exceed that minimum, e.g., by being a large epilog. Leave the assert
specifying the hard requirement of maximum fragment size.
  • Loading branch information
BruceForstall committed Sep 14, 2023
1 parent 9101b85 commit 385247e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/coreclr/jit/emit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3143,7 +3143,7 @@ void emitter::emitSplit(emitLocation* startLoc,
} // end for loop

splitIfNecessary();
assert(curSize < maxSplitSize);
assert(curSize < UW_MAX_FRAGMENT_SIZE_BYTES);
}

/*****************************************************************************
Expand Down

0 comments on commit 385247e

Please sign in to comment.