This is an archive of the discontinued LLVM Phabricator instance.

Fix test breakage caused by r374424
ClosedPublic

Authored by lawrence_danna on Oct 14 2019, 11:36 AM.

Details

Summary

The build directory name is based on the test method name, so having
two test methods with the same name in the same test file is a
problem, even if they're in different test classes.

On linux and darwin this conflict can go unnoticed, but windows
has different filesystem semantics and it will fail when one
process tries to delete files still held open by another.

The problem is fixed just by changing the name of one of the test
methods.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2019, 11:36 AM
This revision is now accepted and ready to land.Oct 14 2019, 11:41 AM
This revision was automatically updated to reflect the committed changes.