[PATCH] D125474: [DWARFLinker][NFC] Add None value to the DwarfLinkerAccelTableKind enum.

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere added inline comments.


================
Comment at: llvm/lib/DWARFLinker/DWARFLinker.cpp:1793
+  case DwarfLinkerAccelTableKind::None:
+    // nothing to do.
+    break;
----------------
Nit: llvm style guide says comments need to be sentences starting with an uppercase letter.


================
Comment at: llvm/tools/dsymutil/dsymutil.cpp:209-227
+static Expected<DwarfLinkerAccelTableKind>
+getAccelTableKind(opt::InputArgList &Args) {
   if (opt::Arg *Accelerator = Args.getLastArg(OPT_accelerator)) {
     StringRef S = Accelerator->getValue();
     if (S == "Apple")
-      return AccelTableKind::Apple;
+      return DwarfLinkerAccelTableKind::Apple;
     if (S == "Dwarf")
----------------
Should we update this and allow `None` as a valid input for dsymutil? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125474