This is an archive of the discontinued LLVM Phabricator instance.

Test: Make inferred-framework-case work on case-insensitive FS.
AbandonedPublic

Authored by sas on Jan 20 2015, 6:02 PM.

Download Raw Diff

Details

Reviewers

compnerd
benlangmuir

Summary

This test fails when run from linux on an NFS share exported by Mac OS.

Diff Detail

Event Timeline

sas updated this revision to Diff 18474.Jan 20 2015, 6:02 PM

sas retitled this revision from to Test: Make inferred-framework-case work on case-insensitive FS..

sas updated this object.

sas edited the test plan for this revision. (Show Details)

sas added a reviewer: compnerd.

sas added a subscriber: Unknown Object (MLST).

sas added a reviewer: benlangmuir.Jan 20 2015, 6:03 PM

compnerd accepted this revision.Jan 22 2015, 3:48 PM

compnerd edited edge metadata.

compnerd added inline comments.

test/Modules/inferred-framework-case.m
6	Might be nicer to make it more explicit that this shouldn't be present, but this works just as well.

This revision is now accepted and ready to land.Jan 22 2015, 3:48 PM

This test is explicitly for checking the case-sensitivity of modules. Changing it to 'Mudule' defeats the purpose :-) I think this test just found a bug!

This revision now requires changes to proceed.Jan 22 2015, 3:56 PM

Ah, interesting. I hadn't realized that this was specifically for case sensitivity issues.

Revision Contents

Path

Size

test/

Modules/

inferred-framework-case.m

2 lines

Diff 18474

test/Modules/inferred-framework-case.m

	// RUN: rm -rf %t			// RUN: rm -rf %t
	// RUN: %clang_cc1 -fmodules-cache-path=%t -fmodules -F %S/Inputs %s -verify -DA			// RUN: %clang_cc1 -fmodules-cache-path=%t -fmodules -F %S/Inputs %s -verify -DA
	// FIXME: PR20299 - getCanonicalName() is not implemented on Windows.			// FIXME: PR20299 - getCanonicalName() is not implemented on Windows.
	// REQUIRES: shell			// REQUIRES: shell

	@import MOdule; // expected-error{{module 'MOdule' not found}}			@import Mudule; // expected-error{{module 'Mudule' not found}}
				compnerdUnsubmitted Not Done Reply Inline Actions Might be nicer to make it more explicit that this shouldn't be present, but this works just as well. compnerd: Might be nicer to make it more explicit that this shouldn't be present, but this works just as…
	@import Module;			@import Module;