[llvm] [llvm-gsymutil] Don't warn about non-increasing line tables with merged functions (PR #122217)

Ellis Hoag via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 15:03:40 PST 2025


================
@@ -405,7 +405,7 @@ static void convertFunctionLineTable(OutputAggregator &Out, CUInfo &CUI,
           OS << "warning: duplicate line table detected for DIE:\n";
           Die.dump(OS, 0, DIDumpOptions::getForSingleDIE());
         });
-      else
+      else if (!Gsym.getUseMergedFunctions())
----------------
ellishg wrote:

We really should have brackets around these if/else blocks

https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements

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


More information about the llvm-commits mailing list