tree dd57edcfe60605585a0a96bad1f99052390c9dcd
parent 616b7e54bd9c630379ca0dbcd2de09b76b43b825
author Geoffrey Martin-Noble <gcmn@google.com> 1580252409 -0800
committer Copybara-Service <copybara-worker@google.com> 1580252480 -0800

Bazel to CMake: use explicit kwargs

Make kwargs in bazel to cmake functions explicit. This is more robust and less fiddly. There's some question of whether we should mirror the default values from the .bzl file here or rely on the cmake rule implementation having the right defaults. I went with the former for now because the only downside is redundant specifications in the cmake file, which seems not bad. We might want to remove the defaults in cmake if we go to fully generated files, since then it's one less place to keep in sync.

Tested:
Ran this in preview mode across the entire project and didn't see any new errors
```
$ ./build_tools/scripts/bazel_to_cmake.py --preview | grep -i error
  Reason: `NameError: name 'platform_trampoline_deps' is not defined`
    "status_errors.h"
    "status_win32_errors.h"
    "status_errors.cc"
    "status_win32_errors.cc"
  Reason: `NameError: name 'iree_flatbuffer_cc_library' is not defined`
  Reason: `NameError: name 'PLATFORM_VULKAN_TEST_DEPS' is not defined`
  Reason: `NameError: name 'PLATFORM_VULKAN_LOADER_COPTS' is not defined`
  Reason: `KeyError: "No conversion found for target '//third_party/dawn:dawn_headers'"`
  Reason: `NameError: name 'platform_trampoline_deps' is not defined`
  Reason: `KeyError: '@llvm-project//llvm:tablegen'`
  Reason: `NameError: name 'PLATFORM_VULKAN_DEPS' is not defined`
  Reason: `NameError: name 'PLATFORM_VULKAN_TEST_DEPS' is not defined`
  Reason: `KeyError: "No conversion found for target '@dear_imgui'"`
```

Closes https://github.com/google/iree/pull/589

COPYBARA_INTEGRATE_REVIEW=https://github.com/google/iree/pull/589 from GMNGeoffrey:bazel-to-cmake-1-kwarg-cleanup 30fa731a914d195de3fcbd8912f2c0a4f67f90ec
PiperOrigin-RevId: 292020535