Merge pull request swiftlang#36889 from slavapestov/remove-commented-out-brace-stmt-assert

AST: Remove obsolete commented-out assertion in BraceStmt::create()

Author: slavapestov

lib/AST/Stmt.cpp

@@ -155,21 +155,6 @@ BraceStmt *BraceStmt::create(ASTContext &ctx, SourceLoc lbloc,
   assert(std::none_of(elts.begin(), elts.end(),
                       [](ASTNode node) -> bool { return node.isNull(); }) &&
          "null element in BraceStmt");
-  // Uncomment the following after rdar://53254395 is done:
-  //  if (
-  //    !std::is_sorted(
-  //      elts.begin(), elts.end(),
-  //      [&](ASTNode n1, ASTNode n2) {
-  //        return !ctx.SourceMgr.isBeforeInBuffer(n2.getEndLoc(),
-  //                                              n1.getEndLoc());
-  //      })) {
-  //    llvm::errs() << "Brace statement elements out of order: \n";
-  //    for (auto n: elts) {
-  //      llvm::errs() << n.getOpaqueValue() << ": ";
-  //      n.dump(llvm::errs());
-  //    }
-  //    llvm_unreachable("brace elements out of order")
-  //  }
 
   void *Buffer = ctx.Allocate(totalSizeToAlloc<ASTNode>(elts.size()),
                               alignof(BraceStmt));