Page MenuHomePhabricator

[lldb-mi] Correct regex in the symbol-list-lines test
ClosedPublic

Authored by apolyakov on Sep 14 2018, 9:02 AM.

Details

Summary

The test failed in case of compiling a test suite with
gcc (checked versions are 5.2.0 and 7.3.0) because it
adds one more line entry comparing to clang. It doesn't
break the test's logic, so I just added a regex that matches
this case.

Diff Detail

Repository
rL LLVM

Event Timeline

apolyakov created this revision.Sep 14 2018, 9:02 AM

I've created this revision to be sure that you don't think that this approach is some kind of hack.

aprantl accepted this revision.Sep 14 2018, 12:34 PM

Thanks, that looks like a safe change.

This revision is now accepted and ready to land.Sep 14 2018, 12:34 PM
This revision was automatically updated to reflect the committed changes.