Skip to content

Commit

Permalink
remove inline and let the optimizer decide
Browse files Browse the repository at this point in the history
Pretty much the same for WASM size (a few byte smaller without inlining
the wrappers at the call sites). Instead LLVM inlines the factored-out
function back into the wrappers so performance wise there's no
difference.
  • Loading branch information
aqrln committed Feb 19, 2024
1 parent 9c695c2 commit 64d30b9
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions query-engine/query-structure/src/record.rs
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,6 @@ impl Record {
/// Extracts a `SelectionResult` from this `Record`.
/// `field_names`: Database names of the fields contained in this `Record`.
/// `selected_fields`: The selection to extract.
// #[inline]
pub fn extract_selection_result_from_db_name(
&self,
field_names: &[String],
Expand All @@ -193,7 +192,6 @@ impl Record {

/// Extracts a `SelectionResult` from this `Record` using Prisma field names rather than
/// database names.
// #[inline]
pub fn extract_selection_result_from_prisma_name(
&self,
field_names: &[String],
Expand Down

0 comments on commit 64d30b9

Please sign in to comment.