bazel: Don't disable use_header_modules
There were some old reports that this feature doesn't work with
`-fno-rtti` (b/137799263), but nothing breaks if it's not disabled.
Tested this internally, too (cl/578609723).
Bug: b/267498492
Change-Id: I45c3145dab6b37285760a0cc4b065ed592a611bc
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178565
Commit-Queue: Ted Pudlik <tpudlik@google.com>
Reviewed-by: Erik Gilling <konkers@google.com>
diff --git a/pw_build/bazel_internal/pigweed_internal.bzl b/pw_build/bazel_internal/pigweed_internal.bzl
index fbada31..0276fcd 100644
--- a/pw_build/bazel_internal/pigweed_internal.bzl
+++ b/pw_build/bazel_internal/pigweed_internal.bzl
@@ -27,13 +27,6 @@
# Set linkstatic to avoid building .so files.
kwargs["linkstatic"] = True
- kwargs.setdefault("features", [])
-
- # Crosstool--adding this line to features disables header modules, which
- # don't work with -fno-rtti. Note: this is not a command-line argument,
- # it's "minus use_header_modules".
- kwargs["features"].append("-use_header_modules")
-
def _print_platform_impl(_, ctx):
if hasattr(ctx.rule.attr, "constraint_values"):
for cv in ctx.rule.attr.constraint_values: