[llvm] r204579 - SupportTests.LockFileManagerTest: Add assertions for Win32.

Argyrios Kyrtzidis kyrtzidis at apple.com
Mon Mar 24 15:40:09 PDT 2014


On Mar 24, 2014, at 3:35 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:

> On 24 March 2014 18:32, Argyrios Kyrtzidis <kyrtzidis at apple.com> wrote:
>> 
>> On Mar 23, 2014, at 9:08 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:
>> 
>>> Argyrios, is this representative of how the lock manager is used? If
>>> so, it looks like it would not work on windows. If not, could you
>>> please update the test to avoid the ifdefs? For example, you could
>>> create the link to an existing file and then delete that file.
>> 
>> I’m fine with this change, but could someone with access to a windows machine test and commit this ?
> 
> You forgot to attach a patch?

No, I thought it was clear what you were suggesting.

> 
> Cheers,
> Rafael





More information about the llvm-commits mailing list