• Addressing review comment

In D87043#2294531, @Szelethus wrote:

Congrats on your GSoC! Unless I missed it, it seems like you haven't posted your final report on cfe-dev, even though its an amazing looking document with a lot of examples and explanations. I think it would be great to spread the message, its a work to be proud of!

• Fixing minor spacing issue

• Addressing review comments

In D86373#2231605, @NoQ wrote:

This is easier than D86293 because there's no old value in the freshly constructed smart pointer, right? I suspect that you can still re-use a lot of the implementation with D86293, at least partially.

• Refactoring to reuse common duplicated code

• Removing unnecessary notetags.

• Making the note tags prunable

• Addressing review comments

• Adding checkLiveSymbols and review comments changes

• Add assignment to nullptr case

• Using conjureSymbolVal in case of missing inner pointer value

Changes to use conjureSymbolVal if the inner pointer value is missing

• Make exactly single NodeBuilder exists at any given time

• Fix without passing the NodeBuilder

• Addressing review comments

• Updating test with tags from header file

In D84600#2203604, @Szelethus wrote:

Layering violations are a running theme in the analyzer -- CheckerRegistry and the entire MallocChecker fiasco are two glaring examples. Luckily, this isn't a severe case so I wouldn't worry about it much.

I've been following your patches, but them mentors answer way ahead of me and are very thorough :^) Best of luck with the finale!

• Review comment changes

• Changes from review comments

• Addressing review comments

In D83836#2189636, @NoQ wrote:

These patches look like they're done, maybe let's land them?

In D83877#2189637, @NoQ wrote:

These patches look like they're done, maybe let's land them?

• Changes from review comments

• Adding a missed todo

• Addressing review comments
• Enabling commented out tests

Untrack all changing regions in checkRegionChanges