Skip to content

Commit 2f21555

Browse files
committed
revert shader denorm changes
1 parent 3f292b7 commit 2f21555

File tree

2 files changed

+15
-62
lines changed

2 files changed

+15
-62
lines changed

src/nbl/video/CVulkanPhysicalDevice.cpp

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -401,16 +401,6 @@ std::unique_ptr<CVulkanPhysicalDevice> CVulkanPhysicalDevice::create(core::smart
401401
//vulkan12Properties.denormBehaviorIndependence;
402402
//vulkan12Properties.denormBehaviorIndependence;
403403

404-
if (!vulkan12Properties.shaderSignedZeroInfNanPreserveFloat32)
405-
return nullptr;
406-
if (!vulkan12Properties.shaderDenormPreserveFloat32)
407-
return nullptr;
408-
if (!vulkan12Properties.shaderDenormFlushToZeroFloat32)
409-
return nullptr;
410-
if (!vulkan12Properties.shaderRoundingModeRTEFloat32)
411-
return nullptr;
412-
if (!vulkan12Properties.shaderRoundingModeRTZFloat32)
413-
return nullptr;
414404

415405
// descriptor indexing
416406
properties.limits.maxUpdateAfterBindDescriptorsInAllPools = vulkan12Properties.maxUpdateAfterBindDescriptorsInAllPools;
@@ -900,32 +890,6 @@ std::unique_ptr<CVulkanPhysicalDevice> CVulkanPhysicalDevice::create(core::smart
900890
properties.limits.shaderFloat16 = vulkan12Features.shaderFloat16;
901891
if (!vulkan12Features.shaderInt8)
902892
return nullptr;
903-
904-
if (properties.limits.shaderFloat16) {
905-
if (!vulkan12Properties.shaderSignedZeroInfNanPreserveFloat16)
906-
return nullptr;
907-
if (!vulkan12Properties.shaderDenormPreserveFloat16)
908-
return nullptr;
909-
if (!vulkan12Properties.shaderDenormFlushToZeroFloat16)
910-
return nullptr;
911-
if (!vulkan12Properties.shaderRoundingModeRTEFloat16)
912-
return nullptr;
913-
if (!vulkan12Properties.shaderRoundingModeRTZFloat16)
914-
return nullptr;
915-
}
916-
917-
if (properties.limits.shaderFloat64) {
918-
if (!vulkan12Properties.shaderSignedZeroInfNanPreserveFloat64)
919-
return nullptr;
920-
if (!vulkan12Properties.shaderDenormPreserveFloat64)
921-
return nullptr;
922-
if (!vulkan12Properties.shaderDenormFlushToZeroFloat64)
923-
return nullptr;
924-
if (!vulkan12Properties.shaderRoundingModeRTEFloat64)
925-
return nullptr;
926-
if (!vulkan12Properties.shaderRoundingModeRTZFloat64)
927-
return nullptr;
928-
}
929893

930894
if (!vulkan12Features.descriptorIndexing)
931895
return nullptr;

src/nbl/video/device_capabilities/device_limits.json

Lines changed: 15 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -910,85 +910,74 @@
910910
"type": "bool",
911911
"name": "shaderSignedZeroInfNanPreserveFloat32",
912912
"value": true,
913-
"expose": "REQUIRE"
913+
"expose": "DISABLE"
914914
},
915915
{
916916
"type": "bool",
917917
"name": "shaderSignedZeroInfNanPreserveFloat64",
918-
"value": false,
919-
"expose": "DISABLE"
918+
"value": false
920919
},
921920
{
922921
"type": "bool",
923922
"name": "shaderDenormPreserveFloat16",
924-
"value": false,
925-
"expose": "DISABLE"
923+
"value": false
926924
},
927925
{
928926
"type": "bool",
929927
"name": "shaderDenormPreserveFloat32",
930-
"value": true,
931-
"expose": "REQUIRE"
928+
"value": false
932929
},
933930
{
934931
"type": "bool",
935932
"name": "shaderDenormPreserveFloat64",
936-
"value": false,
937-
"expose": "DISABLE"
933+
"value": false
938934
},
939935
{
940936
"type": "bool",
941937
"name": "shaderDenormFlushToZeroFloat16",
942-
"value": false,
943-
"expose": "DISABLE"
938+
"value": false
944939
},
945940
{
946941
"type": "bool",
947942
"name": "shaderDenormFlushToZeroFloat32",
948-
"value": true,
949-
"expose": "REQUIRE"
943+
"value": false
950944
},
951945
{
952946
"type": "bool",
953947
"name": "shaderDenormFlushToZeroFloat64",
954-
"value": false,
955-
"expose": "DISABLE"
948+
"value": false
956949
},
957950
{
958951
"type": "bool",
959952
"name": "shaderRoundingModeRTEFloat16",
960953
"value": false,
961-
"expose": "DISABLE"
954+
"comment": ["ROADMAP2024 but no good support yet"]
962955
},
963956
{
964957
"type": "bool",
965958
"name": "shaderRoundingModeRTEFloat32",
966-
"value": true,
967-
"expose": "REQUIRE"
959+
"value": false,
960+
"comment": ["ROADMAP2024 but no good support yet"]
968961
},
969962
{
970963
"type": "bool",
971964
"name": "shaderRoundingModeRTEFloat64",
972-
"value": false,
973-
"expose": "DISABLE"
965+
"value": false
974966
},
975967
{
976968
"type": "bool",
977969
"name": "shaderRoundingModeRTZFloat16",
978-
"value": false,
979-
"expose": "DISABLE"
970+
"value": false
980971
},
981972
{
982973
"type": "bool",
983974
"name": "shaderRoundingModeRTZFloat32",
984-
"value": true,
985-
"expose": "REQUIRE"
975+
"value": false
986976
},
987977
{
988978
"type": "bool",
989979
"name": "shaderRoundingModeRTZFloat64",
990-
"value": false,
991-
"expose": "DISABLE"
980+
"value": false
992981
}
993982
]
994983
},

0 commit comments

Comments
 (0)