[Lldb-commits] [PATCH] D29909: Break some more dependencies in lldbUtility

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 13 15:04:38 PST 2017


zturner added a comment.

`git log --follow` seems to understand.  Is that sufficient to guarantee that it will be retained on the SVN side?

  D:\src\llvm-mono>git log --follow lldb/source/Target/ModuleCache.cpp
  commit 07bf36c627f3a4304dbb8ea3f8347110a025bd93
  Author: Zachary Turner <zturner at google.com>
  Date:   Mon Feb 13 13:21:39 2017 -0800
  
      Break some more dependencies in lldbUtility.
  
  commit 2b9af9a9efe04af53a63e56f35dd835c814f000a
  Author: Mehdi Amini <mehdi.amini at apple.com>
  Date:   Fri Nov 11 04:29:25 2016 +0000
  
      Prevent at compile time converting from Error::success() to Expected<T>
  
      This would trigger an assertion at runtime otherwise.
  
      Differential Revision: https://reviews.llvm.org/D26482
  
  commit 990debc1558714a5eda436d1e9bb4786b55bb2d5
  Author: Mehdi Amini <mehdi.amini at apple.com>
  Date:   Fri Nov 11 04:28:40 2016 +0000
  
      Make the Error class constructor protected
  
      This is forcing to use Error::success(), which is in a wide majority
      of cases a lot more readable.
  
      Differential Revision: https://reviews.llvm.org/D26481


https://reviews.llvm.org/D29909





More information about the lldb-commits mailing list