Move `countLines()` inside if pragma

This function was recognized as unused for MSVC and arm builds.
diff --git a/c++/src/kj/async-coroutine-test.c++ b/c++/src/kj/async-coroutine-test.c++
index 0375bc3..a30d374 100644
--- a/c++/src/kj/async-coroutine-test.c++
+++ b/c++/src/kj/async-coroutine-test.c++
@@ -378,6 +378,7 @@
       awaitPromise(kj::mv(rejectedThrowyDtorPromise)).wait(waitScope));
 }
 
+#if (!_MSC_VER || defined(__clang__)) && !__aarch64__
 uint countLines(StringPtr s) {
   uint lines = 0;
   for (char c: s) {
@@ -386,7 +387,6 @@
   return lines;
 }
 
-#if (!_MSC_VER || defined(__clang__)) && !__aarch64__
 // TODO(msvc): This test relies on GetFunctorStartAddress, which is not supported on MSVC currently,
 //   so skip the test.
 // TODO(someday): Test is flakey on arm64, depending on how it's compiled. I haven't had a chance to