[PATCH] D113825: [llvm-readobj][XCOFF] dump auxiliary symbols.
Nico Weber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 12 06:16:53 PST 2022
thakis added inline comments.
================
Comment at: llvm/test/tools/llvm-readobj/XCOFF/symbols-invalid.test:6
+
+# CASE1: llvm-readobj: warning: '[[FILE]]': the non-function C_EXT symbol at index 1 should have only 1 auxiliary entry, i.e. the CSECT auxiliary entry
+
----------------
jhenderson wrote:
> The problem is a missing `{{(.exe)?}}` after the tool name. Usually, we don't include tool names in the check, and just check from "warning:" onwards.
>
> Applies throughout this test.
Hm, normally we don't print "toolname.exe: " in diags on Windows, but just "toolname: ". We usually also don't print the full path to the tool but just the basename.
Maybe the Right Fix is to fix the diag output in llvm-readobj instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113825/new/
https://reviews.llvm.org/D113825
More information about the llvm-commits
mailing list