[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