romange / helio

A modern framework for backend development based on io_uring Linux interface
Apache License 2.0
435 stars 49 forks source link

fix: prevent compiler optimizations related to loading stale tls object #178

Closed romange closed 10 months ago

romange commented 10 months ago

Specifically, during the function stack migration between threads we may encounter faulty compiler optimizations in the release mode. See https://stackoverflow.com/questions/75592038/how-to-disable-clang-expression-elimination-for-thread-local-variable for more details.

codecov-commenter commented 10 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (792bd8c) 78.38% compared to head (078bc98) 78.40%.

Files Patch % Lines
util/fibers/listener_interface.cc 90.00% 2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #178 +/- ## ========================================== + Coverage 78.38% 78.40% +0.01% ========================================== Files 98 98 Lines 7305 7311 +6 ========================================== + Hits 5726 5732 +6 Misses 1579 1579 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.