[PATCH] D69428: [GlobalOpt] Remove valgrind specific hacks (revert r160529)

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 13 22:32:10 PDT 2021


mehdi_amini added a comment.

> On the other hand, all three leak checkers that we have tried don't cope with the transformation well at all

Well, again it depends on your point of view: from my point of view they work perfectly well here and the problem could very well be seen as to be fixed in the user code if this is what they intend (otherwise it will find more unintended leak potentially).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69428



More information about the llvm-commits mailing list