Skip to content

Conversation

waterlens
Copy link
Collaborator

Copy link

Consider documenting target platform requirements

Category
Maintainability
Code Snippet
"monotonic_clock_native.mbt": ["native", "llvm"]
Recommendation
Add a comment or documentation explaining why both 'native' and 'llvm' targets are needed and what functionality each provides
Reasoning
Future maintainers may not understand why both targets are necessary. Having clear documentation about the purpose of each target platform helps with long-term maintenance and prevents accidental removal of required targets.

@Young-Flash Young-Flash merged commit 5a1a678 into llvm_backend Apr 9, 2025
8 of 12 checks passed
@Young-Flash Young-Flash deleted the fix-clock-on-llvm branch April 9, 2025 09:11
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