Repository navigation
Add xcc700 and rcc700 compiler apps - #52
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughAdds two self-hosting mini C compiler apps. xcc700 emits Xtensa code, and rcc700 emits RV32 code. Each app includes SDK and component configuration, a Tactility wrapper for command-line arguments and file flags, app metadata, and documentation. The compiler implementations parse supported C constructs and generate ELF output. The change also updates the repository license statement and the icon font used by a list-view unit test. Priority: ⬇️ Low Merge Risk: 🟡 Moderate · up to Long tokens in source files can corrupt compiler memory or disrupt compilation in either new app. Add bounds checks before merging. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 19.47% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 113 functions across 5 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
ℹ️ Review info
⚙️ Run configuration
- Configuration used: Organization UI
- Review profile: CHILL
- Plan: Advanced
- Run ID:
6ffca81a-72c6-4dc5-93ec-75fbba026ce5
📒 Files selected for processing (14)
Apps/Rcc700/CMakeLists.txtApps/Rcc700/LICENSEApps/Rcc700/README.mdApps/Rcc700/main/CMakeLists.txtApps/Rcc700/main/Source/rcc700.cApps/Rcc700/main/Source/tactility.cApps/Rcc700/manifest.propertiesApps/Xcc700/CMakeLists.txtApps/Xcc700/LICENSEApps/Xcc700/README.mdApps/Xcc700/main/CMakeLists.txtApps/Xcc700/main/Source/tactility.cApps/Xcc700/main/Source/xcc700.cApps/Xcc700/manifest.properties
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
|
@valdanylchuk If you are ever in the neighbourhood of Amsterdam, I owe you a coffee (at the very least). |
Summary by CodeRabbit
.elfextension.