Replies: 1 comment
-
yeah inline usage could def use a closer look cc @DaniPopes |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I think we don't need to add
#[inline]
forprivate function
andpub(crate) function
, because LLVM generally will inline every private function that is needs inlining.we just need to mark public functions with #[inline] to enable cross-crate inlining. This will result in bloat if used incorrectly and might result in slower execution time.
Some material:
Beta Was this translation helpful? Give feedback.
All reactions