Adding tensorflow/core/platform/default/BUILD and tensorflow/core/platform/windows/BUILD.

This is part of the refactoring described in the Tensorflow Build Improvements RFC: tensorflow/community#179
Subsequent changes will migrate targets from build_refactor.bzl into the new BUILD files.

PiperOrigin-RevId: 284712709
Change-Id: I650eb200ba0ea87e95b15263bad53b0243732ef5

Author: bmzhao

tensorflow/BUILD

@@ -478,7 +478,7 @@ bzl_library(
     visibility = ["//visibility:public"],
     deps = [
         "//tensorflow/core/platform:build_config_root_bzl",
-        "//tensorflow/core/platform:cuda_build_defs_bzl",
+        "//tensorflow/core/platform/default:cuda_build_defs_bzl",
         "//third_party/mkl:build_defs_bzl",
         "//third_party/mkl_dnn:build_defs_bzl",
         "//third_party/ngraph:build_defs_bzl",

tensorflow/compiler/tf2xla/BUILD

@@ -1,6 +1,6 @@
 load("//tensorflow:tensorflow.bzl", "tf_cc_binary", "tf_cc_test", "tf_cuda_cc_test")
 load(
-    "//tensorflow/core/platform:default/cuda_build_defs.bzl",
+    "//tensorflow/core/platform/default:cuda_build_defs.bzl",
     "if_cuda_is_configured",
 )
 load(

tensorflow/compiler/xla/service/BUILD

@@ -9,7 +9,7 @@ load(
 )
 load("//tensorflow:tensorflow.bzl", "tf_cc_test")
 load(
-    "//tensorflow/core/platform:default/cuda_build_defs.bzl",
+    "//tensorflow/core/platform/default:cuda_build_defs.bzl",
     "if_cuda_is_configured",
 )
 load(

tensorflow/compiler/xla/service/gpu/BUILD

@@ -17,7 +17,7 @@ load(
 )
 load("@local_config_cuda//cuda:build_defs.bzl", "if_cuda")
 load(
-    "//tensorflow/core/platform:default/cuda_build_defs.bzl",
+    "//tensorflow/core/platform/default:cuda_build_defs.bzl",
     "if_cuda_is_configured",
 )
 load(

tensorflow/core/BUILD

@@ -874,8 +874,6 @@ cc_library(
         "//tensorflow/core/lib/bfloat16:bfloat16.h",
         "//tensorflow/core/platform:byte_order.h",
         "//tensorflow/core/platform:cpu_info.h",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/logging.h",
         "//tensorflow/core/platform:dynamic_annotations.h",
         "//tensorflow/core/platform:macros.h",
         "//tensorflow/core/platform:mutex.h",
@@ -885,6 +883,8 @@ cc_library(
         "//tensorflow/core/platform:thread_annotations.h",
         "//tensorflow/core/platform:tstring.h",
         "//tensorflow/core/platform:types.h",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:logging.h",
     ],
     visibility = ["//visibility:public"],
     deps =
@@ -2379,13 +2379,13 @@ cc_library(
     name = "tflite_portable_logging",
     hdrs = [
         "//tensorflow/core/lib/bfloat16:bfloat16.h",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/logging.h",
         "//tensorflow/core/platform:logging.h",
         "//tensorflow/core/platform:macros.h",
         "//tensorflow/core/platform:platform.h",
         "//tensorflow/core/platform:tstring.h",
         "//tensorflow/core/platform:types.h",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:logging.h",
     ],
     copts = tf_copts(),
     linkopts = ["-ldl"],
@@ -2408,8 +2408,6 @@ cc_library(
         "lib/jpeg/jpeg_mem.h",
         "//tensorflow/core/lib/bfloat16:bfloat16.h",
         "//tensorflow/core/lib/core:legacy_lib_core_stringpiece_header",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/logging.h",
         "//tensorflow/core/platform:dynamic_annotations.h",
         "//tensorflow/core/platform:logging.h",
         "//tensorflow/core/platform:macros.h",
@@ -2418,6 +2416,8 @@ cc_library(
         "//tensorflow/core/platform:stringpiece.h",
         "//tensorflow/core/platform:tstring.h",
         "//tensorflow/core/platform:types.h",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:logging.h",
     ],
     copts = tf_copts(),
     linkopts = ["-ldl"],
@@ -2443,15 +2443,15 @@ cc_library(
         "//tensorflow/core/lib/bfloat16:bfloat16.h",
         "//tensorflow/core/lib/core:legacy_lib_core_stringpiece_header",
         "//tensorflow/core/lib/gtl:legacy_android_gif_internal_headers",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/logging.h",
         "//tensorflow/core/platform:dynamic_annotations.h",
         "//tensorflow/core/platform:logging.h",
         "//tensorflow/core/platform:macros.h",
         "//tensorflow/core/platform:mem.h",
         "//tensorflow/core/platform:platform.h",
         "//tensorflow/core/platform:tstring.h",
         "//tensorflow/core/platform:types.h",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:logging.h",
     ],
     copts = tf_copts(),
     linkopts = ["-ldl"],

tensorflow/core/kernels/BUILD

@@ -21,7 +21,7 @@ load("//tensorflow:tensorflow.bzl", "if_nccl")
 load("//tensorflow:tensorflow.bzl", "tf_cuda_cc_test")
 load("//tensorflow:tensorflow.bzl", "tf_cuda_cc_tests")
 load(
-    "//tensorflow/core/platform:default/cuda_build_defs.bzl",
+    "//tensorflow/core/platform/default:cuda_build_defs.bzl",
     "if_cuda_is_configured",
 )
 load(

tensorflow/core/kernels/rnn/BUILD

@@ -7,7 +7,7 @@ load(
     "tf_kernel_library",
 )
 load(
-    "//tensorflow/core/platform:default/cuda_build_defs.bzl",
+    "//tensorflow/core/platform/default:cuda_build_defs.bzl",
     "if_cuda_is_configured",
 )
 load(

tensorflow/core/platform/BUILD

@@ -18,7 +18,7 @@ load(
     "tf_protobuf_deps",
 )
 load(
-    "//tensorflow/core/platform:default/build_refactor.bzl",
+    "//tensorflow/core/platform/default:build_refactor.bzl",
     "tf_instantiate_platform_libraries",
     "tf_legacy_srcs_no_runtime_google",
     "tf_logging_deps",
@@ -890,31 +890,31 @@ filegroup(
     name = "legacy_srcs_no_runtime",
     srcs = [
         ":legacy_srcs_common",
-        "//tensorflow/core/platform:default/context.h",
-        "//tensorflow/core/platform:default/cord.h",
-        "//tensorflow/core/platform:default/dynamic_annotations.h",
-        "//tensorflow/core/platform:default/env.cc",
-        "//tensorflow/core/platform:default/human_readable_json.cc",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/load_library.cc",
-        "//tensorflow/core/platform:default/logging.h",
-        "//tensorflow/core/platform:default/monitoring.cc",
-        "//tensorflow/core/platform:default/mutex.h",
-        "//tensorflow/core/platform:default/mutex_data.h",
-        "//tensorflow/core/platform:default/net.cc",
-        "//tensorflow/core/platform:default/notification.h",
-        "//tensorflow/core/platform:default/port.cc",
-        "//tensorflow/core/platform:default/posix_file_system.cc",
-        "//tensorflow/core/platform:default/posix_file_system.h",
-        "//tensorflow/core/platform:default/stacktrace.h",
-        "//tensorflow/core/platform:default/stacktrace_handler.cc",
-        "//tensorflow/core/platform:default/strong_hash.h",
-        "//tensorflow/core/platform:default/subprocess.cc",
-        "//tensorflow/core/platform:default/subprocess.h",
-        "//tensorflow/core/platform:default/tracing.cc",
-        "//tensorflow/core/platform:default/tracing_impl.h",
-        "//tensorflow/core/platform:default/unbounded_work_queue.cc",
-        "//tensorflow/core/platform:default/unbounded_work_queue.h",
+        "//tensorflow/core/platform/default:context.h",
+        "//tensorflow/core/platform/default:cord.h",
+        "//tensorflow/core/platform/default:dynamic_annotations.h",
+        "//tensorflow/core/platform/default:env.cc",
+        "//tensorflow/core/platform/default:human_readable_json.cc",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:load_library.cc",
+        "//tensorflow/core/platform/default:logging.h",
+        "//tensorflow/core/platform/default:monitoring.cc",
+        "//tensorflow/core/platform/default:mutex.h",
+        "//tensorflow/core/platform/default:mutex_data.h",
+        "//tensorflow/core/platform/default:net.cc",
+        "//tensorflow/core/platform/default:notification.h",
+        "//tensorflow/core/platform/default:port.cc",
+        "//tensorflow/core/platform/default:posix_file_system.cc",
+        "//tensorflow/core/platform/default:posix_file_system.h",
+        "//tensorflow/core/platform/default:stacktrace.h",
+        "//tensorflow/core/platform/default:stacktrace_handler.cc",
+        "//tensorflow/core/platform/default:strong_hash.h",
+        "//tensorflow/core/platform/default:subprocess.cc",
+        "//tensorflow/core/platform/default:subprocess.h",
+        "//tensorflow/core/platform/default:tracing.cc",
+        "//tensorflow/core/platform/default:tracing_impl.h",
+        "//tensorflow/core/platform/default:unbounded_work_queue.cc",
+        "//tensorflow/core/platform/default:unbounded_work_queue.h",
     ],
     visibility = ["//tensorflow/core:__pkg__"],
 )
@@ -967,6 +967,14 @@ filegroup(
     visibility = ["//tensorflow/core:__pkg__"],
 )
 
+bzl_library(
+    name = "build_config_root_bzl",
+    srcs = [
+        "build_config_root.bzl",
+        "//tensorflow/core/platform/default:build_config_root.bzl",
+    ],
+)
+
 # TODO(gunan): Remove the following once references in core/BUILD is removed.
 exports_files(
     glob(
@@ -988,19 +996,6 @@ exports_files(
     ),
 )
 
-bzl_library(
-    name = "build_config_root_bzl",
-    srcs = [
-        "build_config_root.bzl",
-        "default/build_config_root.bzl",
-    ],
-)
-
-bzl_library(
-    name = "cuda_build_defs_bzl",
-    srcs = ["default/cuda_build_defs.bzl"],
-)
-
 exports_files(
     [
         "stacktrace.h",

tensorflow/core/platform/build_config.bzl

@@ -1,7 +1,7 @@
 """Provides a redirection point for platform specific implementations of starlark utilities."""
 
 load(
-    "//tensorflow/core/platform:default/build_config.bzl",
+    "//tensorflow/core/platform/default:build_config.bzl",
     _pyx_library = "pyx_library",
     _tf_additional_all_protos = "tf_additional_all_protos",
     _tf_additional_binary_deps = "tf_additional_binary_deps",

tensorflow/core/platform/build_config_root.bzl

@@ -1,7 +1,7 @@
 """Provides a redirection point for platform specific implementations of starlark utilities."""
 
 load(
-    "//tensorflow/core/platform:default/build_config_root.bzl",
+    "//tensorflow/core/platform/default:build_config_root.bzl",
     _if_dynamic_kernels = "if_dynamic_kernels",
     _if_static = "if_static",
     _if_static_and_not_mobile = "if_static_and_not_mobile",

tensorflow/core/platform/default/BUILD

@@ -0,0 +1,46 @@
+# Tensorflow default + linux implementations of tensorflow/core/platform libraries.
+load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
+
+package(
+    default_visibility = [
+        "//tensorflow/core/platform:__pkg__",
+    ],
+    licenses = ["notice"],  # Apache 2.0
+)
+
+bzl_library(
+    name = "cuda_build_defs_bzl",
+    srcs = ["cuda_build_defs.bzl"],
+    visibility = ["//tensorflow:__subpackages__"],
+)
+
+package_group(
+    name = "core_and_platform_packages",
+    packages = [
+        "//tensorflow/core",
+        "//tensorflow/core/platform",
+    ],
+)
+
+exports_files(
+    srcs = glob(
+        ["*"],
+        exclude = [
+            "integral_types.h",
+            "logging.h",
+            "test.cc",
+            "test_benchmark.cc",
+        ],
+    ),
+    visibility = ["//tensorflow/core/platform:__pkg__"],
+)
+
+exports_files(
+    srcs = [
+        "integral_types.h",
+        "logging.h",
+        "test.cc",
+        "test_benchmark.cc",
+    ],
+    visibility = [":core_and_platform_packages"],
+)

tensorflow/core/platform/default/build_config.bzl

@@ -554,30 +554,30 @@ def tf_platform_srcs(files):
 
 def tf_additional_lib_hdrs():
     return [
-        "//tensorflow/core/platform:default/context.h",
-        "//tensorflow/core/platform:default/cord.h",
-        "//tensorflow/core/platform:default/dynamic_annotations.h",
-        "//tensorflow/core/platform:default/integral_types.h",
-        "//tensorflow/core/platform:default/logging.h",
-        "//tensorflow/core/platform:default/mutex.h",
-        "//tensorflow/core/platform:default/mutex_data.h",
-        "//tensorflow/core/platform:default/notification.h",
-        "//tensorflow/core/platform:default/stacktrace.h",
-        "//tensorflow/core/platform:default/strong_hash.h",
-        "//tensorflow/core/platform:default/test_benchmark.h",
-        "//tensorflow/core/platform:default/tracing_impl.h",
-        "//tensorflow/core/platform:default/unbounded_work_queue.h",
+        "//tensorflow/core/platform/default:context.h",
+        "//tensorflow/core/platform/default:cord.h",
+        "//tensorflow/core/platform/default:dynamic_annotations.h",
+        "//tensorflow/core/platform/default:integral_types.h",
+        "//tensorflow/core/platform/default:logging.h",
+        "//tensorflow/core/platform/default:mutex.h",
+        "//tensorflow/core/platform/default:mutex_data.h",
+        "//tensorflow/core/platform/default:notification.h",
+        "//tensorflow/core/platform/default:stacktrace.h",
+        "//tensorflow/core/platform/default:strong_hash.h",
+        "//tensorflow/core/platform/default:test_benchmark.h",
+        "//tensorflow/core/platform/default:tracing_impl.h",
+        "//tensorflow/core/platform/default:unbounded_work_queue.h",
     ] + select({
         "//tensorflow:windows": [
-            "//tensorflow/core/platform:windows/intrinsics_port.h",
-            "//tensorflow/core/platform:windows/stacktrace.h",
-            "//tensorflow/core/platform:windows/subprocess.h",
-            "//tensorflow/core/platform:windows/wide_char.h",
-            "//tensorflow/core/platform:windows/windows_file_system.h",
+            "//tensorflow/core/platform/windows:intrinsics_port.h",
+            "//tensorflow/core/platform/windows:stacktrace.h",
+            "//tensorflow/core/platform/windows:subprocess.h",
+            "//tensorflow/core/platform/windows:wide_char.h",
+            "//tensorflow/core/platform/windows:windows_file_system.h",
         ],
         "//conditions:default": [
-            "//tensorflow/core/platform:default/posix_file_system.h",
-            "//tensorflow/core/platform:default/subprocess.h",
+            "//tensorflow/core/platform/default:posix_file_system.h",
+            "//tensorflow/core/platform/default:subprocess.h",
         ],
     })
 
@@ -629,8 +629,8 @@ def tf_additional_test_deps():
 
 def tf_additional_test_srcs():
     return [
-        "default/test.cc",
-        "default/test_benchmark.cc",
+        "//tensorflow/core/platform/default:test.cc",
+        "//tensorflow/core/platform/default:test_benchmark.cc",
     ]
 
 def tf_kernel_tests_linkstatic():