Remove CCP from size and performance recipes, pending bugfixes
authorDavid Neto <dneto@google.com>
Fri, 5 Jan 2018 18:21:36 +0000 (13:21 -0500)
committerDavid Neto <dneto@google.com>
Fri, 5 Jan 2018 19:01:18 +0000 (14:01 -0500)
Currently CCP is incorrectly optimizing loops.
See https://github.com/KhronosGroup/SPIRV-Tools/issues/1143

source/opt/optimizer.cpp

index 6c8b854..b6c4c18 100644 (file)
@@ -125,7 +125,9 @@ Optimizer& Optimizer::RegisterPerformancePasses() {
       .RegisterPass(CreateLocalSingleStoreElimPass())
       .RegisterPass(CreateInsertExtractElimPass())
       .RegisterPass(CreateLocalMultiStoreElimPass())
-      .RegisterPass(CreateCCPPass())
+      // TODO(dneto): Disable CCP until it optimizes loops correctly
+      // https://github.com/KhronosGroup/SPIRV-Tools/issues/1143
+      //.RegisterPass(CreateCCPPass())
       .RegisterPass(CreateAggressiveDCEPass())
       .RegisterPass(CreateDeadBranchElimPass())
       .RegisterPass(CreateBlockMergePass())
@@ -147,7 +149,9 @@ Optimizer& Optimizer::RegisterSizePasses() {
       .RegisterPass(CreateLocalSingleStoreElimPass())
       .RegisterPass(CreateInsertExtractElimPass())
       .RegisterPass(CreateLocalMultiStoreElimPass())
-      .RegisterPass(CreateCCPPass())
+      // TODO(dneto): Disable CCP until it optimizes loops correctly
+      // https://github.com/KhronosGroup/SPIRV-Tools/issues/1143
+      //.RegisterPass(CreateCCPPass())
       .RegisterPass(CreateAggressiveDCEPass())
       .RegisterPass(CreateDeadBranchElimPass())
       .RegisterPass(CreateBlockMergePass())