[all-commits] [llvm/llvm-project] 8043be: [JSONNodeDumper] Do not print mangled names for lo...

Nikita Popov via All-commits all-commits at lists.llvm.org
Wed Dec 22 23:55:59 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8043beb8901cb95fce4266b498262acf08494510
      https://github.com/llvm/llvm-project/commit/8043beb8901cb95fce4266b498262acf08494510
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2021-12-23 (Thu, 23 Dec 2021)

  Changed paths:
    M clang/lib/AST/JSONNodeDumper.cpp
    M clang/test/AST/ast-dump-decl-json.c
    M clang/test/AST/ast-dump-decl-json.m
    M clang/test/AST/ast-dump-expr-json.c
    M clang/test/AST/ast-dump-expr-json.cpp
    M clang/test/AST/ast-dump-expr-json.m
    M clang/test/AST/ast-dump-funcs-json.cpp
    M clang/test/AST/ast-dump-if-json.cpp
    M clang/test/AST/ast-dump-stmt-json.c
    M clang/test/AST/ast-dump-stmt-json.cpp
    M clang/test/AST/ast-dump-stmt-json.m
    M clang/test/AST/ast-dump-template-decls-json.cpp

  Log Message:
  -----------
  [JSONNodeDumper] Do not print mangled names for local variables (PR49111)

Mangled names are not meaningful for variables with local storage,
and may not be well defined (getting the mangled name for VLA
crashes the mangler). As such, do not include them in the JSON
dump.

This allows running update_cc_test_checks on some OpenMP tests again.

Fixes https://github.com/llvm/llvm-project/issues/49111.

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




More information about the All-commits mailing list