Re: [Lldb-commits] [PATCH] D13602: [LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Interpreter and Host; other minor fixes.

2015-10-22 Thread Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251054: Fix Clang-tidy modernize-use-override warnings in some files in… (authored by eugenezelenko). Changed prior to commit: http://reviews.llvm.org/D13602?vs=36981=38163#toc Repository: rL LLVM

Re: [Lldb-commits] [PATCH] D13602: [LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Interpreter and Host; other minor fixes.

2015-10-22 Thread Greg Clayton via lldb-commits
clayborg accepted this revision. clayborg added a comment. Looks good. I would venture to say just checkin any clang tidy things that aren't changing code functionality without need for review. Repository: rL LLVM http://reviews.llvm.org/D13602

[Lldb-commits] [PATCH] D13602: [LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Interpreter and Host; other minor fixes.

2015-10-09 Thread Eugene Zelenko via lldb-commits
Eugene.Zelenko created this revision. Eugene.Zelenko added reviewers: brucem, labath, clayborg. Eugene.Zelenko added a subscriber: lldb-commits. Eugene.Zelenko set the repository for this revision to rL LLVM. This is second attempt based on current code. I checked this patch on my own build on