[llvm-commits] [PATCH] CrashRecoveryContext.cpp Win32 support

[NAKAMURA Takumi]

Aaron and Anton,

Aaron's patch is good at functionality but a little redundant due to
considering w2k.

Anton, I propose it would be the time for us to abandon Windows 2000
to coming llvm 3.0.
It was 20th Century System :D
How do you think?

Aaron, do you think your patch would be simpler if we could get rid of w2k?
(oc, I think so)

...Takumi


2011/8/8 Aaron Ballman <aaron at aaronballman.com>:
> Hello!  I noticed there was a "fixme" listed in
> CrashRecoveryContext.cpp and that some Win32 support was needed.  I
> believe this patch will solve the issue.  I have tested it with Visual
> Studio 2010, and MinGW gcc 4.5.2 and the behavior is the same as what
> I get on OS X.
>
> I was unable to get the test suite to run in my environment (had all
> the prerequisites installed, followed the instructions on the site),
> so I've not been able to run this against the suite.
>
> One thing to note about this patch is that is only provides support
> for Windows XP and higher.  Trying to support older versions of
> Windows is possible (Win2k and higher) but comes with more pitfalls
> (as noted in the comments).
>
> If you have any questions, please ask!
>
> ~Aaron
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>