cxa-throw: Don't auto-detect#458
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughOn Linux the Meson build now sets ChangesLinux CXA throw wiring
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
a9dd4d6 to
82e906f
Compare
82e906f to
84a8cbe
Compare
This speeds up running meson (re)configuration from 16.6s to 2.9s. More importantly though, it's better to just specify that we expect this to work rather than try to compile something, since that can silently mask errors and disable this feature.
84a8cbe to
2e8c23b
Compare
cole-h
left a comment
There was a problem hiding this comment.
I was really proud of this autodetection 😭
But I do like speed, so 🚀
Motivation
This speeds up running meson (re)configuration from 16.6s to 2.9s.
More importantly though, it's better to just specify that we expect this to work rather than try to compile something, since that can silently mask errors and disable this feature.
Context
Summary by CodeRabbit