> 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 reasonable to me, thanks!
>>> When you commit
> 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 the tests are still
>> run
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 goog
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
http://lists.llvm.org/cgi-bin/mailman/listin
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.
https:
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
http://lis
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: lit
