[PATCH] D111416: [Demangle] Add support for D simple basic and compound types

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 13 10:58:07 PDT 2021


MaskRay added inline comments.


================
Comment at: llvm/lib/Demangle/DLangDemangle.cpp:449
+  switch (*Mangled) {
+  case 'O': /* shared(T) */
+    Mangled++;
----------------
Prefer `//` for C++ (https://llvm.org/docs/CodingStandards.html#comment-formatting)




================
Comment at: llvm/lib/Demangle/DLangDemangle.cpp:450
+  case 'O': /* shared(T) */
+    Mangled++;
+    Decl->append("shared(");
----------------
Prefer pre-increment `++Mangled` (https://llvm.org/docs/CodingStandards.html#prefer-preincrement)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111416



More information about the llvm-commits mailing list