[PATCH] D127171: [DWARF] Support 'G' in dwarf parser

Florian Mayer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 13:20:26 PDT 2022


fmayer added inline comments.


================
Comment at: llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp:1144
+          case 'G':
+            break;
           }
----------------
MaskRay wrote:
> Can reuse `break` attached to `case 'B'`
Is that actually worth saving one line for potential confusion down the line if anyone were ever to add anything to one of those?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127171/new/

https://reviews.llvm.org/D127171



More information about the llvm-commits mailing list