Skip to content

Build native Preview link plans and runtime policy - #3364

Draft
karim-alweheshy wants to merge 4 commits into
preview-review-05-link-responsesfrom
preview-review-09-link-policy
Draft

karim-alweheshy wants to merge 4 commits into
preview-review-05-link-responsesfrom
preview-review-09-link-policy

Conversation

@karim-alweheshy

@karim-alweheshy karim-alweheshy commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Add processing for native static-library link inputs and a helper that plans runtime linker arguments. Preserve literal static-provider values and ordinary shell-quoted arguments as separate contracts. Native Preview activation remains a later change.

Stack

Based on #3361. The updated parent is incorporated and the parser conflict is resolved. Land #3361 first, then reconcile and revalidate this branch against main; do not merge into the feature base.

Validation

Local processor tests passed (46), as did runtime-planning tests (28). The runtime-helper Bazel target executed and passed; the processor target reused a cached result. Scoped Buildifier and whitespace checks passed. Generated-example and Canvas behavior were not tested.

Testing

Run bazel test //tools/params_processors:link_params_processor_tests //test/internal/bazel_integration_files:preview_runtime_link_params.

Karim Alweheshy added 3 commits September 23, 2026 20:58
Add the static-library response mode, parse driver policy into declared metadata, and keep ordinary-link behavior covered by its existing tests.

Signed-off-by: Karim Alweheshy <karim.alweheshy@reddit.com>
Query the selected Clang driver for its Preview runtime closure and validate the bounded response contract with unit tests.

Signed-off-by: Karim Alweheshy <karim.alweheshy@reddit.com>
Signed-off-by: Karim Alweheshy <karim.alweheshy@reddit.com>
@karim-alweheshy
karim-alweheshy requested a review from a team October 1, 2026 18:00
Signed-off-by: Karim Alweheshy <karim.alweheshy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant