Changeset 661e7b0 for src/Common/CodeLocation.hpp

Timestamp:

Aug 13, 2024, 11:54:04 AM (2 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

2870cb6, 4558df2

Parents:

4941716

Message:

After a years (or at least half a year) the CodeLocation? optimization is merged in. Added Symbol (using Racket's name for interned strings), and used it for CodeLocation? file names. The optimizes for the high number of copies - both instances with the same value and copy operations - and consistently brings down runtime by a few percent.

File:

• src/Common/CodeLocation.hpp (modified) (3 diffs)

src/Common/CodeLocation.hpp

@@ -17 +17 @@
 
 #include <iostream>
-#include <string>
+#include "Symbol.hpp"
 
 struct CodeLocation {
         int first_line = -1, first_column = -1, last_line = -1, last_column = -1;
-        std::string filename = "";
+        Symbol filename = "";
 
         /// Create a new unset CodeLocation.
@@ -46 +46 @@
 
         bool startsBefore( CodeLocation const & other ) const {
-                if( filename < other.filename ) return true;
-                if( filename > other.filename ) return false;
+                if( filename.str() < other.filename.str() ) return true;
+                if( filename.str() > other.filename.str() ) return false;
 
                 if( first_line < other.first_line ) return true;
@@ -72 +72 @@
 inline std::ostream & operator<<( std::ostream & out, const CodeLocation & location ) {
         // Column number ":1" allows IDEs to parse the error message and position the cursor in the source text.
-        return location.isSet() ? out << location.filename << ":" << location.first_line << ":1 " : out;
+        return location.isSet() ? out << location.filename.str() << ":" << location.first_line << ":1 " : out;
 }