[PATCH] D136397: [Clang] Change AnonStructIds in MangleContext to per-function based

2022-10-23 Thread Rong Xu via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. xur marked an inline comment as done. Closed by commit rG6cee5393371f: [Clang] Change AnonStructIds in MangleContext to per-function based (authored by xur). Herald

[PATCH] D136397: [Clang] Change AnonStructIds in MangleContext to per-function based

2022-10-21 Thread Rong Xu via Phabricator via cfe-commits
xur marked an inline comment as done. xur added inline comments. Comment at: clang/include/clang/AST/Mangle.h:97 + +// If FunctionDecl is passed in, the anonnousstructID will be per-function +// based. rsmith wrote: > Fixed. Thanks! CHANGES SINCE LAST

[PATCH] D136397: [Clang] Change AnonStructIds in MangleContext to per-function based

2022-10-20 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Looks good to me. Comment at: clang/include/clang/AST/Mangle.h:97 + +// If FunctionDecl is passed in, the anonnousstructID will be per-function +// based.

[PATCH] D136397: [Clang] Change AnonStructIds in MangleContext to per-function based

2022-10-20 Thread Rong Xu via Phabricator via cfe-commits
xur created this revision. xur added reviewers: rsmith, davidxl. Herald added a subscriber: wenlei. Herald added a project: All. xur requested review of this revision. Clang is generating different mangled names for the same lambda function in build that are slightly different (like from