[PATCH] D122820: [GH54588]Fix ItaniumMangler for NTTP unnamed unions w/ unnamed structs
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 4 06:54:06 PDT 2022
thakis added a comment.
> Oh hm, if it's due to the wrong llvm-cxxfilt being called, maybe we just need to update clang/test/lit.cfg.py to add llvm-cxxfilt to `tools` to make sure it uses the just-built one instead of the one on PATH (? not sure).
Actually, I consistently saw this failing on all our mac bots, and as far as I can tell they _don't_ have llvm-cxxfilt on PATH. Maybe it's due to macOS prepending a `_` by default and `llvm-cxxfilt` insisting on host underscoriness?
% out/gn/bin/llvm-cxxfilt _Z4funcPci
_Z4funcPci
% out/gn/bin/llvm-cxxfilt __Z4funcPcia
func(char*, int, signed char)
Yes, looks like it. Since you're always passing a linux triple (which doesn't add the extra underscore), I think it'll work if you pass `-n` to llvm-cxxfilt to force that mode:
% out/gn/bin/llvm-cxxfilt -n _Z4funcPcia
func(char*, int, signed char)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122820/new/
https://reviews.llvm.org/D122820
More information about the cfe-commits
mailing list