[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
  % 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)

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list