[Lldb-commits] [PATCH] D64536: Adding inline comments to code view type records

2019-07-16 Thread NILANJANA BASU via Phabricator via lldb-commits
nilanjana_basu updated this revision to Diff 210142. nilanjana_basu added a comment. Added record kind name in the comments for better readability Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64536/new/ https://reviews.llvm.org/D64536 Files: llvm/include/llvm/

[Lldb-commits] [PATCH] D64536: Adding inline comments to code view type records

2019-07-16 Thread NILANJANA BASU via Phabricator via lldb-commits
nilanjana_basu updated this revision to Diff 210123. nilanjana_basu added a comment. Corrected comments for codeview type record from "NumAttributes" to "AccessSpecifier" Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64536/new/ https://reviews.llvm.org/D64536 Fi

[Lldb-commits] [PATCH] D64536: Adding inline comments to code view type records

2019-07-11 Thread NILANJANA BASU via Phabricator via lldb-commits
nilanjana_basu marked 2 inline comments as done. nilanjana_basu added inline comments. Comment at: llvm/include/llvm/DebugInfo/CodeView/CodeViewRecordIO.h:63 - Error mapInteger(TypeIndex &TypeInd); + Error mapInteger(TypeIndex &TypeInd, const char* Comment = nullptr); -

[Lldb-commits] [PATCH] D64536: Adding inline comments to code view type records

2019-07-11 Thread NILANJANA BASU via Phabricator via lldb-commits
nilanjana_basu updated this revision to Diff 209278. nilanjana_basu added a comment. Herald added subscribers: lldb-commits, abidh. Herald added a project: LLDB. String parameter passing is done using Twine instead of using C strings, as suggested in the review Repository: rLLDB LLDB CHANGES