[PATCH] D97472: [test] Use host platform specific error message substitution in lit tests

Abhina Sree via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 3 11:06:36 PST 2021


abhina.sreeskantharajan added a comment.

In D97472#2600526 <https://reviews.llvm.org/D97472#2600526>, @jdenny wrote:

> I may be lost.  Why is `--ignore-case` still needed in `llvm/test/tools/yaml2obj/ELF/DWARF/debug-gnu-pubnames.yaml`?  I thought the special case for win32 was supposed to handle that.

I missed this testcase in my old patch. I noticed it because it started to fail when I tried to add --ignore-case as a solution. I can fix it in this patch as well if desired or open a separate patch for it after this one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97472



More information about the llvm-commits mailing list