[all-commits] [llvm/llvm-project] eaca93: [Clang][CodeGen]Fix __builtin_dump_struct missing ...

wangyihan via All-commits all-commits at lists.llvm.org
Thu Apr 28 22:22:16 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: eaca933c59fd61b3df4697b5fae0eeec67acfaeb
      https://github.com/llvm/llvm-project/commit/eaca933c59fd61b3df4697b5fae0eeec67acfaeb
  Author: jonasyhwang <jonasyhwang at tencent.com>
  Date:   2022-04-29 (Fri, 29 Apr 2022)

  Changed paths:
    M clang/lib/CodeGen/CGBuiltin.cpp
    M clang/test/CodeGen/dump-struct-builtin.c

  Log Message:
  -----------
  [Clang][CodeGen]Fix __builtin_dump_struct missing record type field name

Thanks for @rsmith to point this. I'm sorry for introducing this bug.
See @rsmith 's comment in https://reviews.llvm.org/D122248
Eg:(By @rsmith ) https://godbolt.org/z/o7vcbWaEf
I have added a test case
struct:
```
struct U19A {
    int a;
  };
  struct U19B {
    struct U19A a;
  };

  struct U19B a = {
    .a.a = 2022
  };
```
Dump result:
```
struct U19B {
    struct U19A a = {
        int a = 2022
    }
}
```

Reviewed By: erichkeane

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




More information about the All-commits mailing list