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.

Sign up for GitHub

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

Jump to bottom

precompile some headers #164

Merged
merged 6 commits into from
Dec 10, 2024
Merged

precompile some headers #164

merged 6 commits into from
Dec 10, 2024

Conversation

AndreyMlashkin
Copy link
Collaborator

No description provided.

@martun martun self-requested a review November 29, 2024 19:53
@AndreyMlashkin AndreyMlashkin marked this pull request as draft November 30, 2024 07:07
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch 3 times, most recently from d5ab6bf to 15b5db4 Compare December 4, 2024 06:25
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 4, 2024 •
edited
Loading

Clang Test Results

  150 files  ± 0    150 suites  ±0   26m 0s ⏱️ + 7m 19s
3 030 tests  - 93  3 024 ✅  - 93  6 💤 ±0  0 ❌ ±0 
3 092 runs   - 93  3 086 ✅  - 93  6 💤 ±0  0 ❌ ±0 

Results for commit 23a13b9. ± Comparison against base commit c458ac8.

This pull request removes 99 and adds 6 tests. Note that renamed tests count towards both. 
pack_equal_test_suite ‑ bubb_to_bubb_1
pack_equal_test_suite ‑ bubb_to_bubb_2
pack_equal_test_suite ‑ bubb_to_bubb_3
pack_equal_test_suite ‑ bubb_to_bulb_1
pack_equal_test_suite ‑ bubb_to_bulb_2
pack_equal_test_suite ‑ bubb_to_lubb_1
pack_equal_test_suite ‑ bubb_to_lubb_2
pack_equal_test_suite ‑ bubb_to_lubb_3
pack_equal_test_suite ‑ bubb_to_lulb_1
pack_equal_test_suite ‑ bubb_to_lulb_2
…

pack_imploder_test_suite ‑ be_to_be_1
pack_imploder_test_suite ‑ be_to_be_2
pack_imploder_test_suite ‑ be_to_le_1
pack_imploder_test_suite ‑ be_to_le_2
pack_imploder_test_suite ‑ be_to_le_3
pack_imploder_test_suite ‑ bubb_to_lulb_4

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 4, 2024 •
edited
Loading

Gcc Test Results

  150 files   - 1    150 suites   - 1   21m 20s ⏱️ - 21m 39s
3 123 tests  - 3  3 117 ✅  - 3  6 💤 ±0  0 ❌ ±0 
3 185 runs   - 3  3 179 ✅  - 3  6 💤 ±0  0 ❌ ±0 

Results for commit 23a13b9. ± Comparison against base commit c458ac8.

This pull request removes 3 tests. 
polynomial_dfs_benchmark_test_suite ‑ dummy_test
polynomial_dfs_benchmark_test_suite ‑ polynomial_product_test
polynomial_dfs_benchmark_test_suite ‑ polynomial_sum_real_test

♻️ This comment has been updated with latest results.

@AndreyMlashkin AndreyMlashkin marked this pull request as ready for review December 4, 2024 12:22
@martun martun self-requested a review December 4, 2024 14:48
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch from e6d6379 to b974f83 Compare December 5, 2024 09:16
@martun martun self-requested a review December 5, 2024 09:19
@AndreyMlashkin AndreyMlashkin added this pull request to the merge queue Dec 5, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Dec 5, 2024
akokoshn
akokoshn reviewed Dec 5, 2024
View reviewed changes
flake.nix Show resolved Hide resolved
parallel-crypto3/libs/parallel-math/test/CMakeLists.txt Outdated Show resolved Hide resolved
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch 2 times, most recently from 8b7c883 to dea2880 Compare December 6, 2024 14:07
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch 2 times, most recently from bf0ed72 to 4868e7b Compare December 9, 2024 14:00
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch from ea84d0b to b439fff Compare December 10, 2024 09:53
@AndreyMlashkin AndreyMlashkin force-pushed the precompile_some_headers branch from 7a20638 to 23a13b9 Compare December 10, 2024 11:22
@martun martun self-requested a review December 10, 2024 11:27
@AndreyMlashkin AndreyMlashkin added this pull request to the merge queue Dec 10, 2024
Merged via the queue into master with commit 0ced39c Dec 10, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akokoshn akokoshn akokoshn left review comments

@martun martun martun approved these changes

Assignees

@AndreyMlashkin AndreyMlashkin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AndreyMlashkin @martun @akokoshn