[clang] [clang] Properly set file and line info for -ftime-trace (PR #106277)

Ilya Biryukov via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 28 02:29:47 PDT 2024


================
@@ -223,15 +223,15 @@ Frontend (test.cc)
 | | | | EvaluateAsRValue (<test.cc:8:21, col:25>)
 | | | EvaluateAsBooleanCondition (<test.cc:8:21, col:25>)
 | | | | EvaluateAsRValue (<test.cc:8:21, col:25>)
-| ParseDeclarationOrFunctionDefinition (test.cc:16:1)
+| ParseDeclarationOrFunctionDefinition (test.cc:16)
 | | ParseFunctionDefinition (slow_test)
 | | | EvaluateAsInitializer (slow_value)
 | | | EvaluateAsConstantExpr (<test.cc:17:33, col:59>)
----------------
ilya-biryukov wrote:

Is there any reason to keep these locations different?
Why not don't we change them into the structured output for consistency too?

https://github.com/llvm/llvm-project/pull/106277


More information about the cfe-commits mailing list