Skip to content

Add options for compiling in sanitiser, profiling, and valgrind support for better debugging #68

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
Open
wants to merge 92 commits into
base: main
Choose a base branch
from
Open

Add options for compiling in sanitiser, profiling, and valgrind support for better debugging #68

wants to merge 92 commits into from

Conversation

@Bidski
Copy link
Contributor

@Bidski Bidski commented Jun 21, 2023

No description provided.

Bidski and others added 30 commits June 6, 2023 11:51
@Bidski
Replace volatile bools with std::atomic<bool>s
a3a10d4
@Bidski
Add sanitisation compile options
e07753d
@Bidski
Add colourised compiler output
fb37432
@Bidski
Replace volatile bools with std::atomic<bool>s
24a7848
@Bidski
Add unit test for sync priority queue
619ee7c
@Bidski
Add missing include
4aa971d
@Bidski
Add extra source annotations to improve valgrind support
76526bd
@Bidski
Add missing includes
b76b397
@Bidski
Replace volatile bools with std::atomic<bool>s
75a98f6
@Bidski
Lock mutex before calling notify_all()
69480c4
@Bidski
Use lock_guard
a65fe4c
@Bidski
Store true rather than false
b7a590a
@Bidski
Merge remote-tracking branch 'origin/main' into biddulph/stuff
b3a8525
@Bidski
Add missing includes
0d6722a
@Bidski
Add a Group fusion word
2b90f12
@Bidski
Add a Pool fusion word
8e15607
@Bidski
Update CallbackGenerator to use pool and group
f3e69e3
@Bidski
Rewrite TaskScheduler to handle everything
0d4fd91
@Bidski
Add missing Pool word to Reactor
6243f00
@Bidski
Add an immediate flag to ReactionTask for Direct emits
1f7101a
@Bidski
Rewrite Always to suit new TaskScheduler
bfa865f
@Bidski
Change Initialise emits to an emit with no delay
6f5e82b
@Bidski
Change MainThread to use the main thread pool
56ee199
@Bidski
Implement Pool DSL
6a1a774
@Bidski
#undef valgrind symbols before setting them
ee85a92
@TrentHouliston
Remove reschedule
7a3c4ef
@TrentHouliston
formatting
daa6ce9
@Bidski
Merge remote-tracking branch 'origin/main' into biddulph/stuff
35729a2
@Bidski
Apply clang-tidy fixes
7eb0215
@Bidski
Make Always work
0b9c0d8
@Bidski Bidski requested a review from TrentHouliston June 21, 2023 05:49
@Bidski Bidski changed the base branch from biddulph/stuff to main June 23, 2023 04:47
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

TrentHouliston added a commit that referenced this pull request Aug 14, 2024
@TrentHouliston
Add options for enabling sanitisers, Partially replaces #68
5a0488a
@TrentHouliston TrentHouliston mentioned this pull request Aug 14, 2024
Merged
TrentHouliston added a commit that referenced this pull request Aug 14, 2024
@TrentHouliston
Add options for enabling sanitisers (#119)
4640ec3 
Partially replaces #68

#68 had extra intrusive options for valgrind which this leaves out.
#68 is left open in case we need those in the future
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TrentHouliston TrentHouliston Awaiting requested review from TrentHouliston

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Bidski @TrentHouliston