Merge pull request #75488 from nate-chandler/test/20240725/1

[Test] Replaced hyphens with underscores in SIL unit test names.

Author: nate-chandler

include/swift/SIL/Test.h

@@ -148,7 +148,7 @@ class FunctionTest final {
   /// To create a new test, just write
   ///
   ///     namespace swift::test {
-  ///     static FunctionTest myTest("my-test", [](
+  ///     static FunctionTest myTest("my_new_test", [](
   ///       SILFunction &function, Arguments &arguments, FunctionTest &test){
   ///         // test code
   ///       });

lib/SIL/IR/SILInstruction.cpp

@@ -126,7 +126,7 @@ void SILInstruction::moveBefore(SILInstruction *Later) {
 }
 
 namespace swift::test {
-FunctionTest MoveBeforeTest("instruction-move-before",
+FunctionTest MoveBeforeTest("instruction_move_before",
                             [](auto &function, auto &arguments, auto &test) {
                               auto *inst = arguments.takeInstruction();
                               auto *other = arguments.takeInstruction();

lib/SIL/IR/SILType.cpp

@@ -1310,7 +1310,7 @@ namespace swift::test {
 // - SILValue: value
 // Dumps:
 // - message
-static FunctionTest IsSILTrivial("is-sil-trivial", [](auto &function,
+static FunctionTest IsSILTrivial("is_sil_trivial", [](auto &function,
                                                       auto &arguments,
                                                       auto &test) {
   SILValue value = arguments.takeValue();

lib/SIL/IR/SILValue.cpp

@@ -162,7 +162,7 @@ namespace swift::test {
 // Dumps:
 // - value
 // - whether it's lexical
-static FunctionTest IsLexicalTest("is-lexical", [](auto &function,
+static FunctionTest IsLexicalTest("is_lexical", [](auto &function,
                                                    auto &arguments,
                                                    auto &test) {
   auto value = arguments.takeValue();

lib/SIL/IR/ValueOwnership.cpp

@@ -725,7 +725,7 @@ namespace swift::test {
 // - SILValue: value
 // Dumps:
 // - message
-static FunctionTest GetOwnershipKind("get-ownership-kind", [](auto &function,
+static FunctionTest GetOwnershipKind("get_ownership_kind", [](auto &function,
                                                               auto &arguments,
                                                               auto &test) {
   SILValue value = arguments.takeValue();

lib/SIL/Utils/FieldSensitivePrunedLiveness.cpp

@@ -1212,7 +1212,7 @@ namespace swift::test {
 // locations. In that case, the def nodes may be stores and the uses may be
 // destroy_addrs.
 static FunctionTest FieldSensitiveMultiDefUseLiveRangeTest(
-    "fieldsensitive-multidefuse-liverange",
+    "fieldsensitive_multidefuse_liverange",
     [](auto &function, auto &arguments, auto &test) {
       SmallVector<SILBasicBlock *, 8> discoveredBlocks;
       auto value = arguments.takeValue();

lib/SIL/Utils/OwnershipLiveness.cpp

@@ -381,7 +381,7 @@ namespace swift::test {
 // - function
 // - the computed pruned liveness
 // - the liveness boundary
-static FunctionTest LinearLivenessTest("linear-liveness", [](auto &function,
+static FunctionTest LinearLivenessTest("linear_liveness", [](auto &function,
                                                              auto &arguments,
                                                              auto &test) {
   SILValue value = arguments.takeValue();

lib/SIL/Utils/OwnershipUtils.cpp

@@ -103,7 +103,7 @@ namespace swift::test {
 // - the value
 // - whether it has a pointer escape
 static FunctionTest OwnershipUtilsHasPointerEscape(
-    "has-pointer-escape", [](auto &function, auto &arguments, auto &test) {
+    "has_pointer_escape", [](auto &function, auto &arguments, auto &test) {
       auto value = arguments.takeValue();
       auto has = findPointerEscape(value);
       value->print(llvm::outs());
@@ -2144,7 +2144,7 @@ namespace swift::test {
 // - function
 // - the enclosing defs
 static FunctionTest FindEnclosingDefsTest(
-    "find-enclosing-defs", [](auto &function, auto &arguments, auto &test) {
+    "find_enclosing_defs", [](auto &function, auto &arguments, auto &test) {
       function.print(llvm::outs());
       llvm::outs() << "Enclosing Defs:\n";
       visitEnclosingDefs(arguments.takeValue(), [](SILValue def) {
@@ -2169,7 +2169,7 @@ namespace swift::test {
 // - function
 // - the borrow introducers
 static FunctionTest FindBorrowIntroducers(
-    "find-borrow-introducers", [](auto &function, auto &arguments, auto &test) {
+    "find_borrow_introducers", [](auto &function, auto &arguments, auto &test) {
       function.print(llvm::outs());
       llvm::outs() << "Introducers:\n";
       visitBorrowIntroducers(arguments.takeValue(), [](SILValue def) {
@@ -2288,7 +2288,7 @@ namespace swift::test {
 // - function
 // - the adjacent phis
 static FunctionTest VisitInnerAdjacentPhisTest(
-    "visit-inner-adjacent-phis",
+    "visit_inner_adjacent_phis",
     [](auto &function, auto &arguments, auto &test) {
       function.print(llvm::outs());
       visitInnerAdjacentPhis(cast<SILPhiArgument>(arguments.takeValue()),

lib/SIL/Utils/ScopedAddressUtils.cpp

@@ -113,7 +113,7 @@ namespace swift::test {
 // Dumps:
 // - the liveness result and boundary
 static FunctionTest ScopedAddressLivenessTest(
-    "scoped-address-liveness", [](auto &function, auto &arguments, auto &test) {
+    "scoped_address_liveness", [](auto &function, auto &arguments, auto &test) {
       auto value = arguments.takeValue();
       assert(!arguments.hasUntaken());
       llvm::outs() << "Scoped address analysis: " << value;

lib/SILOptimizer/Analysis/BasicCalleeAnalysis.cpp

@@ -94,7 +94,7 @@ namespace swift::test {
 // Dumps:
 // - instruction
 // - whether it's a deinit barrier
-static FunctionTest IsDeinitBarrierTest("is-deinit-barrier", [](auto &function,
+static FunctionTest IsDeinitBarrierTest("is_deinit_barrier", [](auto &function,
                                                                 auto &arguments,
                                                                 auto &test) {
   auto *instruction = arguments.takeInstruction();