Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix - Missing BPF_MUL result sign extension #518

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

Lichtso
Copy link

@Lichtso Lichtso commented Sep 19, 2023

#498 was split into #494, #499 and some commits left for the old multiplication instruction encodings, which were not PRed yet.

@codecov-commenter
Copy link

Codecov Report

Merging #518 (6778797) into main (d18d47c) will increase coverage by 0.03%.
Report is 11 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #518      +/-   ##
==========================================
+ Coverage   89.60%   89.63%   +0.03%     
==========================================
  Files          23       23              
  Lines       10261    10301      +40     
==========================================
+ Hits         9194     9233      +39     
- Misses       1067     1068       +1     
Files Changed Coverage Δ
src/jit.rs 92.36% <100.00%> (ø)

... and 4 files with indirect coverage changes

@Lichtso Lichtso merged commit ef0264d into main Sep 20, 2023
12 checks passed
@Lichtso Lichtso deleted the fix/bpf_mul_result_sign_extension branch September 20, 2023 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants