Szelethus added a comment. Woah, the code looks amazing, cheers on the refactoring! I'll be honest, I'm struggling a bit with the sentence "we're now in the top frame". In order, I don't understand what does
- we - now - in the top frame mean. "Top-level argument" is another one -- Do we have **precise** definitions for there terms? ================ Comment at: clang/test/Analysis/malloc.cpp:151 + char *getName() { + if (!name) { + name = static_cast<char *>(malloc(10)); ---------------- Is this relevant? `name` will never be null. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60112/new/ https://reviews.llvm.org/D60112 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits