Skip to content

Commit 9a7e33c

Browse files
committed
undo validation test changes
1 parent 2a63db8 commit 9a7e33c

3 files changed

Lines changed: 71 additions & 161 deletions

File tree

tools/clang/unittests/HLSL/CompilerTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3712,7 +3712,7 @@ TEST_F(CompilerTest, CompileWhenODumpThenOptimizerMatch) {
37123712
LPCWSTR Target = L"ps_6_0";
37133713
CreateBlobFromText("float4 main() : SV_Target { return 0; }", &pSource);
37143714

3715-
LPCWSTR Args[3] = {OptLevel, L"-select-validator external", L"/Odump"};
3715+
LPCWSTR Args[2] = {OptLevel, L"/Odump"};
37163716

37173717
// Get the passes for this optimization level.
37183718
VERIFY_SUCCEEDED(pCompiler->Compile(pSource, L"source.hlsl", L"main",

tools/clang/unittests/HLSL/DxilModuleTest.cpp

Lines changed: 26 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -514,67 +514,51 @@ TEST_F(DxilModuleTest, SetValidatorVersion) {
514514
return;
515515

516516
// Current version
517-
VerifyValidatorVersionMatches(L"ps_6_2", {L"-select-validator external"});
518-
VerifyValidatorVersionMatches(L"lib_6_3", {L"-select-validator external"});
517+
VerifyValidatorVersionMatches(L"ps_6_2", {});
518+
VerifyValidatorVersionMatches(L"lib_6_3", {});
519519

520520
// Current version, with validation disabled
521-
VerifyValidatorVersionMatches(L"ps_6_2",
522-
{L"-Vd", L"-select-validator external"});
523-
VerifyValidatorVersionMatches(L"lib_6_3",
524-
{L"-Vd", L"-select-validator external"});
521+
VerifyValidatorVersionMatches(L"ps_6_2", {L"-Vd"});
522+
VerifyValidatorVersionMatches(L"lib_6_3", {L"-Vd"});
525523

526524
// Override validator version
527-
VerifyValidatorVersionMatches(
528-
L"ps_6_2", {L"-validator-version", L"1.2", L"-select-validator external"},
529-
1, 2);
530-
VerifyValidatorVersionMatches(
531-
L"lib_6_3",
532-
{L"-validator-version", L"1.3", L"-select-validator external"}, 1, 3);
525+
VerifyValidatorVersionMatches(L"ps_6_2", {L"-validator-version", L"1.2"}, 1,
526+
2);
527+
VerifyValidatorVersionMatches(L"lib_6_3", {L"-validator-version", L"1.3"}, 1,
528+
3);
533529

534530
// Override validator version, with validation disabled
535-
VerifyValidatorVersionMatches(
536-
L"ps_6_2",
537-
{L"-Vd", L"-validator-version", L"1.2", L"-select-validator external"}, 1,
538-
2);
539-
VerifyValidatorVersionMatches(
540-
L"lib_6_3",
541-
{L"-Vd", L"-validator-version", L"1.3", L"-select-validator external"}, 1,
542-
3);
531+
VerifyValidatorVersionMatches(L"ps_6_2",
532+
{L"-Vd", L"-validator-version", L"1.2"}, 1, 2);
533+
VerifyValidatorVersionMatches(L"lib_6_3",
534+
{L"-Vd", L"-validator-version", L"1.3"}, 1, 3);
543535

544536
// Never can validate (version 0,0):
545-
VerifyValidatorVersionMatches(L"lib_6_1",
546-
{L"-Vd", L"-select-validator external"}, 0, 0);
537+
VerifyValidatorVersionMatches(L"lib_6_1", {L"-Vd"}, 0, 0);
538+
VerifyValidatorVersionMatches(L"lib_6_2", {L"-Vd"}, 0, 0);
547539
VerifyValidatorVersionMatches(L"lib_6_2",
548-
{L"-Vd", L"-select-validator external"}, 0, 0);
549-
VerifyValidatorVersionMatches(
550-
L"lib_6_2",
551-
{L"-Vd", L"-validator-version", L"0.0", L"-select-validator external"}, 0,
552-
0);
540+
{L"-Vd", L"-validator-version", L"0.0"}, 0, 0);
553541
VerifyValidatorVersionMatches(L"lib_6_x", {}, 0, 0);
554-
VerifyValidatorVersionMatches(
555-
L"lib_6_x",
556-
{L"-validator-version", L"0.0", L"-select-validator external"}, 0, 0);
542+
VerifyValidatorVersionMatches(L"lib_6_x", {L"-validator-version", L"0.0"}, 0,
543+
0);
557544

558545
// Failure cases:
559546
VerifyValidatorVersionFails(
560-
L"ps_6_2", {L"-validator-version", L"1.1", L"-select-validator external"},
547+
L"ps_6_2", {L"-validator-version", L"1.1"},
561548
{"validator version 1,1 does not support target profile."});
562549

563550
VerifyValidatorVersionFails(
564-
L"lib_6_2", {L"-Tlib_6_2", L"-select-validator external"},
551+
L"lib_6_2", {L"-Tlib_6_2"},
565552
{"Must disable validation for unsupported lib_6_1 or lib_6_2 targets"});
566553

567-
VerifyValidatorVersionFails(
568-
L"lib_6_2",
569-
{L"-Vd", L"-validator-version", L"1.2", L"-select-validator external"},
570-
{"-validator-version cannot be used with library "
571-
"profiles lib_6_1 or lib_6_2."});
554+
VerifyValidatorVersionFails(L"lib_6_2",
555+
{L"-Vd", L"-validator-version", L"1.2"},
556+
{"-validator-version cannot be used with library "
557+
"profiles lib_6_1 or lib_6_2."});
572558

573-
VerifyValidatorVersionFails(
574-
L"lib_6_x",
575-
{L"-validator-version", L"1.3", L"-select-validator external"},
576-
{"Offline library profile cannot be used with "
577-
"non-zero -validator-version."});
559+
VerifyValidatorVersionFails(L"lib_6_x", {L"-validator-version", L"1.3"},
560+
{"Offline library profile cannot be used with "
561+
"non-zero -validator-version."});
578562
}
579563

580564
TEST_F(DxilModuleTest, PayloadQualifier) {

0 commit comments

Comments
 (0)