[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-13 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL291895: Remove a couple of Stream flags (authored by labath). Changed prior to commit: https://reviews.llvm.org/D28616?vs=84143&id=84267#toc Repository: rL LLVM https://reviews.llvm.org/D28616 Files

[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-12 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. Yeah, I'm starting to remember this a bit. For some reason, we have individual log channels with verbose as a category (LIBLLDB_LOG_VERBOSE, POSIX_LOG_VERBOSE, KDP_LOG_VERBOSE) and we have LLDB_LOG_OPTION_VERBOSE which is set "GetVerbose" is testing for. So you can d

[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-12 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. In https://reviews.llvm.org/D28616#644287, @jingham wrote: > Is this going to make passing LIBLLDB_LOG_OPTION_VERBOSE to > GetLogIfAllCategoriesSet into a "you can't see these logs" operation? > > I use this in a couple places where I have detailed information that I >

[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-12 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. Is this going to make passing LIBLLDB_LOG_OPTION_VERBOSE to GetLogIfAllCategoriesSet into a "you can't see these logs" operation? I use this in a couple places where I have detailed information that I usually don't want to see, but in some odd cases I might need it. Lo

[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-12 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Most of the DWARF stuff is about to go away anyway in favor of using the LLVM DWARF parser as I am currently modifying it to support all we need in LLDB so we can get rid of the entire DWA

[Lldb-commits] [PATCH] D28616: Remove a couple of Stream flags

2017-01-12 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added reviewers: clayborg, zturner. labath added subscribers: lldb-commits, beanz. Herald added a subscriber: aprantl. I came across this while trying to understand what Log::Debug does. It turns out it does not do anything, as there is no instance of someone s