Reduce allocations in Builder::startProfiler.

zeek · Nov 20, 2023 · 79a969c · 79a969c
1 parent dd39e48
commit 79a969c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/hilti/toolchain/include/ast/builder/builder.h b/hilti/toolchain/include/ast/builder/builder.h
@@ -241,7 +241,7 @@ class Builder {
 
     bool empty() const { return _block.statements().empty() && _tmps.empty(); }
 
-    std::optional<Expression> startProfiler(const std::string& name);
+    std::optional<Expression> startProfiler(std::string_view name);
     void stopProfiler(Expression profiler);
 
 private:

diff --git a/hilti/toolchain/src/ast/builder/builder.cc b/hilti/toolchain/src/ast/builder/builder.cc
@@ -107,7 +107,7 @@ void Builder::setLocation(const Location& l) {
     _block._add(statement::SetLocation(builder::string_literal(l.render())));
 }
 
-std::optional<Expression> Builder::startProfiler(const std::string& name) {
+std::optional<Expression> Builder::startProfiler(std::string_view name) {
     if ( ! context()->options().enable_profiling )
         return {};