Re: [PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-08 Thread Adrian Prantl via cfe-commits
> On Sep 8, 2017, at 11:43 AM, Adrian Prantl wrote: > > >> On Sep 8, 2017, at 10:54 AM, Paul Robinson via Phabricator >> wrote: >> >> probinson added a comment. >> >> In https://reviews.llvm.org/D37604#864187, @aprantl wrote: >> >>> This seems

Re: [PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-08 Thread Adrian Prantl via cfe-commits
> On Sep 8, 2017, at 10:54 AM, Paul Robinson via Phabricator > wrote: > > probinson added a comment. > > In https://reviews.llvm.org/D37604#864187, @aprantl wrote: > >> This seems reasonable to me, thanks! >> When you commit this, could you please double-check that

[PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-08 Thread Paul Robinson via Phabricator via cfe-commits
probinson added a comment. In https://reviews.llvm.org/D37604#864187, @aprantl wrote: > This seems reasonable to me, thanks! > When you commit this, could you please double-check that the tests are still > running on the green dragon builders? I'll also keep an eye on them. I was able to

[PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-08 Thread Paul Robinson via Phabricator via cfe-commits
probinson closed this revision. probinson added a comment. r312803. Forgot to put the tag in the commit message https://reviews.llvm.org/D37604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-07 Thread Adrian Prantl via Phabricator via cfe-commits
aprantl accepted this revision. aprantl added a comment. This revision is now accepted and ready to land. This seems reasonable to me, thanks! When you commit this, could you please double-check that the tests are still running on the green dragon builders? I'll also keep an eye on them.

[PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-07 Thread Paul Robinson via Phabricator via cfe-commits
probinson added a comment. The new file goes in the debuginfo-tests directory. It's a separate project so that's probably not obvious from the diff. https://reviews.llvm.org/D37604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37604: Disable debuginfo-tests for non-native configurations

2017-09-07 Thread Paul Robinson via Phabricator via cfe-commits
probinson created this revision. This requires the host triple to match the default target triple, in order to run debuginfo-tests. It's possible this is too restrictive, but offhand it seems like a reasonable starting point. https://reviews.llvm.org/D37604 Files: lit.local.cfg Index: