[PATCH] D98908: Update basic deref API to account for possiblity of free [NFC]
Nuno Lopes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 19 07:03:08 PDT 2021
nlopes added a comment.
Sounds good to me, as long as you commit to removing that cmd switch within a reasonable time frame. Otherwise we accumulate technical debt.
This patch just shows how terrible this LLVM thing of traversing things backwards can be. Just because a pointer can be freed, it doesn't mean it was. This patch is quite pessimistic; essentially it assumes all pointers are dead. What's the plan to avoid regressions?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98908/new/
https://reviews.llvm.org/D98908
More information about the llvm-commits
mailing list