[PATCH] D57094: Fix memory leak in ManagedStatic

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 28 12:45:42 PST 2019

Previous message: [PATCH] D56550: Add recipes for migrating downstream branches of git mirrors
Next message: [PATCH] D57352: [InstCombine] canonicalize cmp/select form of uadd saturate with constant
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

dblaikie added a comment.

Generally (especially bounded) memory allocation that remains reachable but is not allocated isn't considered a leak, as such.

I believe this code as-is is intentional.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57094

Previous message: [PATCH] D56550: Add recipes for migrating downstream branches of git mirrors
Next message: [PATCH] D57352: [InstCombine] canonicalize cmp/select form of uadd saturate with constant
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list