[PATCH] D88940: Add regular expressions to and DWARF Call Frame Information tests in case the architecture specific target is not compiled into LLVM.

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 10:38:19 PDT 2020


clayborg added a comment.

In D88940#2316416 <https://reviews.llvm.org/D88940#2316416>, @labath wrote:

> This basically removes the testing for the original patch, since all of these tests would now pass even if the feature is completely reverted. Fortunately, I think the fix for that is easy. We don't need to add the regex to tests that already presuppose that the relevant llvm target is available. By the looks of things, that's everything except `test/DebugInfo/dwarfdump-debug-frame-simple.test`.

good catch. I have update the patch!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88940/new/

https://reviews.llvm.org/D88940



More information about the llvm-commits mailing list