M-Labs/nac3
M-Labs
/
nac3
8
0
Fork 10
Code Issues 99 Pull Requests 21 Releases Wiki Activity

Minor improvements to multithreaded compilation in Standalone #341

Merged
sb10q merged 1 commits from fix/threading-support into master 2024-08-17 17:37:20 +08:00
Conversation 4 Commits 1 Files Changed 1 +2
derppening commented 2023-10-13 15:04:42 +08:00
Collaborator
Copy Link
  • Require the use of -T to specify threading
  • Treat -T0 as using all available threads
  • Fix run_demo.sh script to clean all previous build output and link all bitcode modules from different threads
- Require the use of `-T` to specify threading - Treat `-T0` as using all available threads - Fix `run_demo.sh` script to clean all previous build output and link all bitcode modules from different threads
derppening added 2 commits 2023-10-13 15:04:45 +08:00
derppening requested review from sb10q 2023-10-13 15:04:50 +08:00
derppening added a new dependency 2023-10-13 15:05:07 +08:00
#298 "with parallel" breakage
derppening removed a dependency 2023-10-13 15:05:10 +08:00
#298 "with parallel" breakage
derppening self-assigned this 2023-10-13 15:05:25 +08:00
derppening added a new dependency 2023-10-13 15:05:35 +08:00
#340 Fix thread safety with LLVM library
derppening commented 2023-10-16 12:22:09 +08:00
Author
Collaborator
Copy Link

v2: Use llvm-link even if there is only one input file

v2: Use `llvm-link` even if there is only one input file
derppening commented 2023-10-16 13:45:54 +08:00
Author
Collaborator
Copy Link

v3: Use nullglob to collect all module*.bc file (even if they do not exist)

v3: Use `nullglob` to collect all `module*.bc` file (even if they do not exist)
derppening commented 2023-10-16 14:19:55 +08:00
Author
Collaborator
Copy Link

v4: Squashed redundant commits

v4: Squashed redundant commits
sb10q reviewed 2023-10-16 15:48:42 +08:00
nac3standalone/demo/run_demo.sh Outdated
@ -52,2 +49,4 @@
clang -c -std=gnu11 -Wall -Wextra -O3 -emit-llvm -o demo.bc demo.c
shopt -s nullglob
llvm-link -o module.bc module*.bc main.bc
sb10q commented 2023-10-16 15:48:42 +08:00
Owner
Copy Link

Isn't module*.bc also matching module.bc? I know you are deleting it above but it looks confusing and a potential source of bugs. Better rename it to something else and clear e.g nac3output.bc ?

Isn't module*.bc also matching module.bc? I know you are deleting it above but it looks confusing and a potential source of bugs. Better rename it to something else and clear e.g nac3output.bc ?
derppening commented 2023-10-16 15:54:37 +08:00
Author
Collaborator
Copy Link

Addressed in v5.

Addressed in v5.
derppening added 2 commits 2023-10-16 15:53:56 +08:00
a50c690428 standalone: Fix run_demo script 
- Link main and module*.bc together if using multiple threads
- Fix temporary files not being deleted
derppening commented 2023-10-16 15:54:41 +08:00
Author
Collaborator
Copy Link

v5: Renamed output bitcode to nac3out.bc.

v5: Renamed output bitcode to `nac3out.bc`.
sb10q merged commit 950f431483 into master 2023-10-16 15:55:56 +08:00
sb10q deleted branch fix/threading-support 2023-10-16 15:55:58 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
sb10q
Labels
Clear labels   
future-extension

   
high-priority

   
low-priority

   
UROP0622
No Label
future-extension
high-priority
low-priority
UROP0622
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
derppening
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Blocks
#340 Fix thread safety with LLVM library
M-Labs/nac3
Reference: M-Labs/nac3#341
No description provided.
Delete Branch "fix/threading-support"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?