[clang] [NFC]Fix memory leak in HeaderSearchTest (PR #95927)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 20 09:22:06 PDT 2024
AaronBallman wrote:
Thank you for the fix! FWIW, I don't think this is NFC as it is changing test behavior (fixing a memory leak); please be sure to get a proper code review for functional changes. The changes are reasonable enough, but one question I have is: why not stack allocate the object rather than heap allocate it?
https://github.com/llvm/llvm-project/pull/95927
More information about the cfe-commits
mailing list