labath added a comment. There wasn't a strong reason for using nullptr, I can convert those to ValueObjectSP(). Given that this was your only objection to this patch, I'm going to assume it is ok to land after this. https://reviews.llvm.org/D35615