[Lldb-commits] [PATCH] D13028: [RFC] Merge dsym and dwarf test cases

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 21 13:20:44 PDT 2015


clayborg added a comment.

I like the idea. One question I have is if tests should opt in to multi-debug, or if tests should opt out. This solution does the opt out method. Would it be better to opt in? With something like:

  @debug_info_test
  def test_stuff(self):
      # ...

I am fine with either, I am just checking in case people have a preference and which one makes more sense. I like opting out because most of our tests will probably need the multiple debug info formats. Also, are we verifying that the compiler supports each different kind of mode? Or do we just try it and we would fail of the compiler say didn't handle DWO debugging?


http://reviews.llvm.org/D13028





More information about the lldb-commits mailing list