[PATCH] Add a few basic tests for llvm-pdbdump
Zachary Turner
zturner at google.com
Fri Feb 20 17:47:08 PST 2015
I had to revert this because of some failing bots.
Examples:
http://lab.llvm.org:8011/builders/llvm-s390x-linux1/builds/13731/steps/test-llvm/logs/stdio
http://lab.llvm.org:8011/builders/llvm-hexagon-elf/builds/15858/steps/test-llvm/logs/stdio
Although there were a few others. Now obviously the test should register as unsupported on those platforms, but they're not. I tested this on both my Linux and my Mac workstations and the tests correctly didn't run due to being unsupported. But I'm not sure how to diagnose the problem further of why they are not getting flagged as unsupported here.
HAVE_DIA_SDK is only set to 1 under an "if (MSVC)" check in the CMake, so I've no idea why it might be getting set here, or if something else might be going on. David, Chandler, any ideas how to figure this out?
REPOSITORY
rL LLVM
http://reviews.llvm.org/D7656
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list