diff --git a/bolt/include/bolt/Passes/BinaryPasses.h b/bolt/include/bolt/Passes/BinaryPasses.h --- a/bolt/include/bolt/Passes/BinaryPasses.h +++ b/bolt/include/bolt/Passes/BinaryPasses.h @@ -142,6 +142,8 @@ /// LT_OPTIMIZE_CACHE piggybacks on the idea from Ispike paper (CGO '04) /// that suggests putting frequently executed chains first in the layout. LT_OPTIMIZE_CACHE, + // CACHE_PLUS and EXT_TSP are synonyms, emit warning of deprecation. + LT_OPTIMIZE_CACHE_PLUS, /// Block reordering guided by the extended TSP metric. LT_OPTIMIZE_EXT_TSP, /// Create clusters and use random order for them. diff --git a/bolt/lib/Passes/BinaryPasses.cpp b/bolt/lib/Passes/BinaryPasses.cpp --- a/bolt/lib/Passes/BinaryPasses.cpp +++ b/bolt/lib/Passes/BinaryPasses.cpp @@ -176,13 +176,21 @@ clEnumValN(bolt::ReorderBasicBlocks::LT_OPTIMIZE_CACHE, "cache", "perform optimal layout prioritizing I-cache " "behavior"), - clEnumValN(bolt::ReorderBasicBlocks::LT_OPTIMIZE_EXT_TSP, "cache+", + clEnumValN(bolt::ReorderBasicBlocks::LT_OPTIMIZE_CACHE_PLUS, "cache+", "perform layout optimizing I-cache behavior"), clEnumValN(bolt::ReorderBasicBlocks::LT_OPTIMIZE_EXT_TSP, "ext-tsp", "perform layout optimizing I-cache behavior"), clEnumValN(bolt::ReorderBasicBlocks::LT_OPTIMIZE_SHUFFLE, "cluster-shuffle", "perform random layout of clusters")), - cl::ZeroOrMore, cl::cat(BoltOptCategory)); + cl::ZeroOrMore, cl::cat(BoltOptCategory), + cl::callback([](const bolt::ReorderBasicBlocks::LayoutType &option) { + if (option == bolt::ReorderBasicBlocks::LT_OPTIMIZE_CACHE_PLUS) { + WithColor::warning() + << "'-reorder-blocks=cache+' is deprecated, " + << "please use '-reorder-blocks=ext-tsp' instead\n"; + ReorderBlocks = bolt::ReorderBasicBlocks::LT_OPTIMIZE_EXT_TSP; + } + })); static cl::opt ReportBadLayout("report-bad-layout", diff --git a/bolt/test/cache+-deprecated.test b/bolt/test/cache+-deprecated.test new file mode 100644 --- /dev/null +++ b/bolt/test/cache+-deprecated.test @@ -0,0 +1,8 @@ +# Check for the replacement of calls to identical functions. + +REQUIRES: system-linux + +RUN: %clangxx %p/Inputs/bolt_icf.cpp -g -Wl,-q -o %t.exe +RUN: llvm-bolt %t.exe -reorder-blocks=cache+ -relocs -o %t 2>&1 | FileCheck %s + +CHECK: '-reorder-blocks=cache+' is deprecated, please use '-reorder-blocks=ext-tsp' instead