Skip to content

Commit

Permalink
Drop the requirement that proto_profile and memprof_profile must be s…
Browse files Browse the repository at this point in the history
…ource files.

PiperOrigin-RevId: 629478677
Change-Id: I57c2cc5a48d6762f9164b0908e9388a835a5e0d7
  • Loading branch information
Googler authored and Copybara-Service committed Apr 30, 2024
1 parent ac5c63b commit ff40449
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions src/main/starlark/builtins_bzl/common/cc/fdo/fdo_profile.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,6 @@ FdoProfileInfo = provider(
)

def _impl(ctx):
# proto_profile and memprof_profile can only be used if at least one of profile
# are provided.
if ctx.file.proto_profile and not ctx.file.proto_profile.is_source:
fail("Attribute proto_profile: the target is not an input file")
if ctx.file.memprof_profile and not ctx.file.memprof_profile.is_source:
fail("Attribute memprof_profile: the target is not an input file")

return FdoProfileInfo(
artifact = ctx.file.profile,
proto_profile_artifact = ctx.file.proto_profile,
Expand Down

0 comments on commit ff40449

Please sign in to comment.