[all-commits] [llvm/llvm-project] 493c85: [clang][NFC] Small mangler cleanups

Nathan Sidwell via All-commits all-commits at lists.llvm.org
Thu Jan 20 11:20:31 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 493c856484015873737d7c995cac9e34101fb9e9
      https://github.com/llvm/llvm-project/commit/493c856484015873737d7c995cac9e34101fb9e9
  Author: Nathan Sidwell <nathan at acm.org>
  Date:   2022-01-20 (Thu, 20 Jan 2022)

  Changed paths:
    M clang/lib/AST/ItaniumMangle.cpp

  Log Message:
  -----------
  [clang][NFC] Small mangler cleanups

In working on a module mangling problem I noticed a few cleanups to the mangler.

1) Use 'if (auto x = ...' idiom in a couple of places.

2) I noticed both 'isFileContext' and 'isNamespace || isTranslationUnit'
   synonyms. Let's use the former.

3) The control flow in the seqId mangling was misordered. Let's channel Count
   von Count. Also fix the inconsistent bracing.

Differential Revision: https://reviews.llvm.org/D117799




More information about the All-commits mailing list