[PATCH] D71202: [test][llvm-cxxfilt] Fix darwin build bot/improve test naming and commenting

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 22:10:05 PST 2019


MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-cxxfilt/strip-underscore-default.test:8
+CHECK: __Z1fv
+CHECK: ba()
----------------
jhenderson wrote:
> compnerd wrote:
> > Can you merge this into the previous test and use -triple to test the defaults?  That ensures that the tests always run and people on other hosts can validate that they do not regress tins things.
> Sounds reasonable. I'll see what I can figure out with that.
The output of sys::getProcessTriple() is fixed (inferred from LLVM_HOST_TRIPLE).

```
static bool shouldStripUnderscore() {
  if (StripUnderscore)
    return true;
  if (NoStripUnderscore)
    return false;
  // If none of them are set, use the default value for platform.
  // macho has symbols prefix with "_" so strip by default.
  return Triple(sys::getProcessTriple()).isOSBinFormatMachO();
}
```

> UNSUPPORTED: system-darwin
> REQUIRES: system-darwin

Seem the best to test the platform differences.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71202





More information about the llvm-commits mailing list