[PATCH] D43627: [DEBUGINFO] Add flag for DWARF2 or less to use sections as references.

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp:274
+  const TargetLoweringObjectFile &TLOF = Asm->getObjFileLowering();
+  if (DD->useSectionsAsReferences()) {
+    LineTableStartSym = TLOF.getDwarfLineSection()->getBeginSymbol();
----------------
ABataev wrote:
> JDevlieghere wrote:
> > The LLVM coding standard says not to put braces around single statement blocks. 
> Only if there is no `else` branch with multiple line substatement. In this case, both substatements must be enclosed in braces. 
But that's not the case here, is it?


Repository:
  rL LLVM

https://reviews.llvm.org/D43627