r210584 - Fix crash with x86_64-pc-win32-macho target. <rdar://problem/17235840>

[Chandler Carruth]

On Wed, Jun 11, 2014 at 4:41 PM, Bob Wilson <bob.wilson at apple.com> wrote:

> > On Jun 10, 2014, at 6:07 PM, David Blaikie <dblaikie at gmail.com> wrote:
> >
> > On Tue, Jun 10, 2014 at 6:04 PM, Bob Wilson <bob.wilson at apple.com>
> wrote:
> >> Thanks for fixing the output to the source tree. I should have caught
> that.
> >>
> >> I don’t especially know what to test for here. I wasn’t involved in
> setting up this win32-macho stuff, and I don’t know much about how it
> works. Apparently whoever did set it up neglected to add any tests. I won’t
> claim this is a good test, but it is far better than what we had before
> (nothing!). If someone who knows about this can provide a better test, then
> that would be great.
> >
> > So how'd you come across the issue? I assume you're /using/ it, in
> > which case it might be useful to add some tests/understand how it
> > works...
>
> No, I’m not personally using it at all. I know there are some people at
> Apple who do. I received a bug report saying that it crashes, so I fixed
> the crash. I agree that it would be very useful to have some good tests,
> but I don’t know enough about it to do that.


Well, if you're supporting the only users of it (and it seems you are)
could you find out how to test it better or find the people know about this?

If there is no one around who knows how to test this stuff, then it is
essentially being left in an unmaintained state. That's no good. If we
really can't find anyone to maintain it, I worry need to rip it out because
it will continue to bit rot. That process seems to have already started as
this crash uncovered.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140611/0c98ae3d/attachment.html>