https://github.com/steakhal commented:
LGTM Thanks.
https://github.com/llvm/llvm-project/pull/68211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DonatNagyE closed
https://github.com/llvm/llvm-project/pull/68211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gamesh411 edited
https://github.com/llvm/llvm-project/pull/68211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gamesh411 approved this pull request.
I can confirm the claims about HeapSpaceRegion being used:
https://github.com/llvm/llvm-project/pull/68211
___
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
This trivial commit removes a 13-year-old FIXME comment from MallocChecker.cpp
because it was fixed at least 10 years ago when `getConjuredHeapSymbolVal()`
was added.
---
Full diff: https://github.com/llvm/llvm-project/pull/68211.diff
1
https://github.com/DonatNagyE created
https://github.com/llvm/llvm-project/pull/68211
This trivial commit removes a 13-year-old FIXME comment from MallocChecker.cpp
because it was fixed at least 10 years ago when `getConjuredHeapSymbolVal()`
was added.
>From 70a4adb3ef1444130ddaf9b04d9d7fb887
