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

[Peter Collingbourne via llvm-commits]

pcc added a comment.

> The problem with adding more run lines is that if I understand this failure correctly, the test would fail if any of the run targets was missing from the build configuration.

Not necessarily, you could have something like REQUIRES: foo-registered-target, bar-registered-target (but would agree this is suboptimal)

> Long term, I think it would be nice if LIT had a way to selectively disable individual run lines based on the build configuration.

Agree


Repository:
  rL LLVM

https://reviews.llvm.org/D24896