[PATCH] D24896: [safestack] Require TargetMachine to be provided.

[Michael LeMay via llvm-commits]

mlemay-intel added a comment.

In https://reviews.llvm.org/D24896#562745, @pcc wrote:

> You could add REQUIRES: x86-registered-target to each of the failing tests


I see that none of the tests in test/Transforms/SafeStack/ currently specify any non-X86 target.  The only tests that do are in arch-specific subdirectories.  So, adding the REQUIRES: x86-registered-target directive could work.  However, I wonder whether it will be necessary in the future to add non-X86 targets to the tests in the base directory.


Repository:
  rL LLVM

https://reviews.llvm.org/D24896