[PATCH] D88329: [objc] Consolidate ObjC name mangle code to AST

Ellis Hoag via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 29 08:16:01 PDT 2020


ellis updated this revision to Diff 295000.
ellis added a comment.

[objc] Fix memory leak in CGObjCMac.cpp

CGObjCMac.cpp was leaking a MangleContext everytime it mangled an ObjC method. We now have an instance variable that allocates and deallocates the context.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88329

Files:
  clang/include/clang/AST/Mangle.h
  clang/lib/AST/ItaniumMangle.cpp
  clang/lib/AST/Mangle.cpp
  clang/lib/AST/MicrosoftMangle.cpp
  clang/lib/CodeGen/CGObjCMac.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88329.295000.patch
Type: text/x-patch
Size: 8624 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200929/11549e76/attachment.bin>


More information about the cfe-commits mailing list