Skip to content

Update actions and readme#51

Open
arthur-qm wants to merge 5 commits intoopen-s4c:dicelessfrom
arthur-qm:diceless
Open

Update actions and readme#51
arthur-qm wants to merge 5 commits intoopen-s4c:dicelessfrom
arthur-qm:diceless

Conversation

@arthur-qm
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Arthur Queiroz Moura <[email protected]>
Copy link
Copy Markdown
Collaborator

@TriedAngle TriedAngle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Comment on lines +35 to +50
- name: Build TSANO Interceptor
run: |
cmake -S dice -B dice/build
cmake --build dice/build --target tsano
- name: Compile Test with Sanitizer
run:
gcc -std=c11 -O1 -g -pthread -fsanitize=thread tests/atomic_stable.c -o tests/atomic_stable
- name: Build Rust Crate (Manual Link)
run: |
cargo build
cargo build --features dice-rs/manual-link
- name: Link the Rust static library (`librecorder.a`) with your standalone dice library (`libdice.a`).
run:
cc -shared -o librecorder.so -Wl,--whole-archive target/debug/librecorder.a target/debug/libdice.a -Wl,--no-whole-archive -lpthread -ldl
- name: Run with TSANO
run: TSANO_LIBDIR=dice/build/deps/tsano ./dice/deps/tsano/tsano LD_PRELOAD=./librecorder.so ./tests/atomic_stable
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Violates DRY, please include the commands only once in the repository to avoid future divergence.

You can check with @DIvanov503 for some good suggestions of how to achieve that

Arthur Queiroz Moura added 3 commits January 27, 2026 14:27
Signed-off-by: Arthur Queiroz Moura <[email protected]>
Signed-off-by: Arthur Queiroz Moura <[email protected]>
Signed-off-by: Arthur Queiroz Moura <[email protected]>
Signed-off-by: Arthur Queiroz Moura <[email protected]>
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.

3 participants