[llvm-bugs] [Bug 34772] New: Fix all remaining testsuite failures with target.experimental.use-modern-type-lookup

via llvm-bugs llvm-bugs at lists.llvm.org
Thu Sep 28 13:19:47 PDT 2017


https://bugs.llvm.org/show_bug.cgi?id=34772

            Bug ID: 34772
           Summary: Fix all remaining testsuite failures with
                    target.experimental.use-modern-type-lookup
           Product: lldb
           Version: unspecified
          Hardware: PC
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: All Bugs
          Assignee: lldb-dev at lists.llvm.org
          Reporter: scallanan at apple.com
                CC: llvm-bugs at lists.llvm.org

Currently target.experimental.use-modern-type-lookup is defaulted to off.

There are several reasons it should be on:

 - ExternalASTMerger is shared code which will allow other clients to complete
types from multiple sources, as LLDB does.

 - This will allow us to remove the ClangASTImporter, a large container of
state which can go bad over time.

 - ExternalASTMerger has much more complete unit testing; I believe the
remaining testsuite failures are because of as-yet-incomplete LLDB use of it.

It's possible to run the testsuite with use-modern-type-lookup enabled, by
changing g_experimental_properties in Target.cpp to make it default-on.  The
resulting testsuite failures need to be fixed and the setting should then be
enabled.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20170928/b27dec53/attachment.html>


More information about the llvm-bugs mailing list