Ensure EoL at the end of trivia to prevent inlining issues

source/parsing/Preprocessor.cpp

@@ -435,6 +435,10 @@ Token Preprocessor::nextRaw() {
         }
     }
 
+    // Ensure EoL at the end of trivia to prevent inlining issues
+    if (trivia.empty() || trivia.back().kind != TriviaKind::EndOfLine)
+        trivia.push_back(Trivia(TriviaKind::EndOfLine, ""sv));
+
     // finally found a real token to return, so update trivia and get out of here
     return token.withTrivia(alloc, trivia.copy(alloc));
 }

tests/unittests/DriverTests.cpp

@@ -241,6 +241,19 @@ TEST_CASE("Driver single-unit parsing") {
     CHECK(driver.reportParseDiags());
 }
 
+TEST_CASE("Driver single-unit parsing files with no EOL") {
+    Driver driver;
+    driver.addStandardArgs();
+
+    auto args = fmt::format("testfoo \"{0}file_with_no_eol.sv\" "
+                            "\"{0}file_uses_define_in_file_with_no_eol.sv\" --single-unit",
+                            findTestDir());
+    CHECK(driver.parseCommandLine(args));
+    CHECK(driver.processOptions());
+    CHECK(driver.parseAllSources());
+    CHECK(driver.reportParseDiags());
+}
+
 TEST_CASE("Driver parsing with library modules") {
     auto guard = OS::captureOutput();
 

tests/unittests/data/file_uses_define_in_file_with_no_eol.sv

@@ -0,0 +1,5 @@
+module main;
+    initial begin
+    	$display("Something: %d", `SOMETHING);
+    end
+endmodule

tests/unittests/data/file_with_no_eol.sv

@@ -0,0 +1 @@
+`define SOMETHING 1337