[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