Author: Ilya Biryukov Date: 2022-05-05T15:04:11Z New Revision: ad2263de9f75119626f316e253fdc1de28d5c23a
URL: https://github.com/llvm/llvm-project/commit/ad2263de9f75119626f316e253fdc1de28d5c23a DIFF: https://github.com/llvm/llvm-project/commit/ad2263de9f75119626f316e253fdc1de28d5c23a.diff LOG: [Sema] Replace invalid FIXME about memory leak. NFC Added in my previous patch by mistake. Added: Modified: clang/lib/Sema/SemaConcept.cpp Removed: ################################################################################ diff --git a/clang/lib/Sema/SemaConcept.cpp b/clang/lib/Sema/SemaConcept.cpp index 4ec37a94089b0..261f57e4d185b 100644 --- a/clang/lib/Sema/SemaConcept.cpp +++ b/clang/lib/Sema/SemaConcept.cpp @@ -341,7 +341,7 @@ bool Sema::CheckConstraintSatisfaction( OutSatisfaction = *Satisfaction; // We cannot use InsertPos here because CheckConstraintSatisfaction might have // invalidated it. - // FIXME: this leaks memory, we should allocate in the arena instead. + // Note that entries of SatisfactionCache are deleted in Sema's destructor. SatisfactionCache.InsertNode(Satisfaction.release()); return false; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
