Skip to content

Commit 13e21bf

Browse files
committed
Update base for Update on "[compiler][hir] Correctly remove non-existent terminal preds when pruning labels"
Missed this initially in `pruneUnusedLabelsHIR`. It wasn't an active bug as `preds` wasn't referenced by later passes, until #30079 [ghstack-poisoned]
2 parents f399ed8 + 133ada7 commit 13e21bf

File tree

60 files changed

+764
-608
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

60 files changed

+764
-608
lines changed

.circleci/config.yml

Lines changed: 2 additions & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ jobs:
281281
at: .
282282
- setup_node_modules
283283
- run: ./scripts/circleci/download_devtools_regression_build.js << parameters.version >> --replaceBuild
284-
- run: node ./scripts/jest/jest-cli.js --build --project devtools --release-channel=experimental --reactVersion << parameters.version >> --ci
284+
- run: node ./scripts/jest/jest-cli.js --build --project devtools --release-channel=experimental --reactVersion << parameters.version >> --ci=circleci
285285

286286
run_devtools_e2e_tests_for_versions:
287287
docker: *docker
@@ -345,31 +345,6 @@ jobs:
345345
yarn extract-errors
346346
git diff --quiet || (echo "Found unminified errors. Either update the error codes map or disable error minification for the affected build, if appropriate." && false)
347347
348-
check_generated_fizz_runtime:
349-
docker: *docker
350-
environment: *environment
351-
steps:
352-
- checkout
353-
- attach_workspace: *attach_workspace
354-
- setup_node_modules
355-
- run:
356-
name: Confirm generated inline Fizz runtime is up to date
357-
command: |
358-
yarn generate-inline-fizz-runtime
359-
git diff --quiet || (echo "There was a change to the Fizz runtime. Run `yarn generate-inline-fizz-runtime` and check in the result." && false)
360-
361-
yarn_test:
362-
docker: *docker
363-
environment: *environment
364-
parallelism: *TEST_PARALLELISM
365-
parameters:
366-
args:
367-
type: string
368-
steps:
369-
- checkout
370-
- setup_node_modules
371-
- run: yarn test <<parameters.args>> --ci
372-
373348
yarn_test_build:
374349
docker: *docker
375350
environment: *environment
@@ -382,7 +357,7 @@ jobs:
382357
- attach_workspace:
383358
at: .
384359
- setup_node_modules
385-
- run: yarn test --build <<parameters.args>> --ci
360+
- run: yarn test --build <<parameters.args>> --ci=circleci
386361

387362
RELEASE_CHANNEL_stable_yarn_test_dom_fixtures:
388363
docker: *docker
@@ -432,42 +407,6 @@ workflows:
432407
build_and_test:
433408
unless: << pipeline.parameters.prerelease_commit_sha >>
434409
jobs:
435-
- check_generated_fizz_runtime:
436-
filters:
437-
branches:
438-
ignore:
439-
- builds/facebook-www
440-
- yarn_test:
441-
filters:
442-
branches:
443-
ignore:
444-
- builds/facebook-www
445-
matrix:
446-
parameters:
447-
args:
448-
# Intentionally passing these as strings instead of creating a
449-
# separate parameter per CLI argument, since it's easier to
450-
# control/see which combinations we want to run.
451-
- "-r=stable --env=development"
452-
- "-r=stable --env=production"
453-
- "-r=experimental --env=development"
454-
- "-r=experimental --env=production"
455-
- "-r=www-classic --env=development --variant=false"
456-
- "-r=www-classic --env=production --variant=false"
457-
- "-r=www-classic --env=development --variant=true"
458-
- "-r=www-classic --env=production --variant=true"
459-
- "-r=www-modern --env=development --variant=false"
460-
- "-r=www-modern --env=production --variant=false"
461-
- "-r=www-modern --env=development --variant=true"
462-
- "-r=www-modern --env=production --variant=true"
463-
- "-r=xplat --env=development --variant=false"
464-
- "-r=xplat --env=development --variant=true"
465-
- "-r=xplat --env=production --variant=false"
466-
- "-r=xplat --env=production --variant=true"
467-
468-
# TODO: Test more persistent configurations?
469-
- '-r=stable --env=development --persistent'
470-
- '-r=experimental --env=development --persistent'
471410
- yarn_build:
472411
filters:
473412
branches:

.github/workflows/compiler-playground.yml renamed to .github/workflows/compiler_playground.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Compiler Playground
1+
name: (Compiler) Playground
22

33
on:
44
push:

.github/workflows/compiler-rust.yml renamed to .github/workflows/compiler_rust.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: React Compiler (Rust)
1+
name: (Compiler) Rust
22

33
on:
44
push:

.github/workflows/compiler-typescript.yml renamed to .github/workflows/compiler_typescript.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: React Compiler (TypeScript)
1+
name: (Compiler) TypeScript
22

33
on:
44
push:

.github/workflows/devtools_check_repro.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: DevTools Check for bug repro
1+
name: (DevTools) Check for bug repro
22
on:
33
issues:
44
types: [opened, edited]

.github/workflows/commit_artifacts.yml renamed to .github/workflows/runtime_commit_artifacts.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Commit Artifacts for Meta WWW and fbsource
1+
name: (Runtime) Commit Artifacts for Meta WWW and fbsource
22

33
on:
44
push:

.github/workflows/runtime_fizz.yml

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
name: (Runtime) Fizz
2+
3+
on:
4+
push:
5+
branches: [main]
6+
pull_request:
7+
paths-ignore:
8+
- 'compiler/**'
9+
10+
jobs:
11+
check_generated_fizz_runtime:
12+
name: Confirm generated inline Fizz runtime is up to date
13+
runs-on: ubuntu-latest
14+
steps:
15+
- uses: actions/checkout@v4
16+
- uses: actions/setup-node@v4
17+
with:
18+
node-version: 18.x
19+
cache: "yarn"
20+
cache-dependency-path: yarn.lock
21+
- name: Restore cached node_modules
22+
uses: actions/cache@v4
23+
id: node_modules
24+
with:
25+
path: "**/node_modules"
26+
key: ${{ runner.arch }}-${{ runner.os }}-modules-${{ hashFiles('yarn.lock') }}
27+
- run: yarn install --frozen-lockfile
28+
- run: |
29+
yarn generate-inline-fizz-runtime
30+
git diff --quiet || (echo "There was a change to the Fizz runtime. Run `yarn generate-inline-fizz-runtime` and check in the result." && false)

.github/workflows/flags.yml renamed to .github/workflows/runtime_flags.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Flags
1+
name: (Runtime) Flags
22

33
on:
44
push:

.github/workflows/flow.yml renamed to .github/workflows/runtime_flow.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Flow
1+
name: (Runtime) Flow
22

33
on:
44
push:
@@ -44,4 +44,4 @@ jobs:
4444
path: "**/node_modules"
4545
key: ${{ runner.arch }}-${{ runner.os }}-modules-${{ hashFiles('yarn.lock') }}
4646
- run: yarn install --frozen-lockfile
47-
- run: node ./scripts/tasks/flow-ci-ghaction ${{ matrix.flow_inline_config_shortname }}
47+
- run: node ./scripts/tasks/flow-ci ${{ matrix.flow_inline_config_shortname }}

.github/workflows/fuzz_tests.yml renamed to .github/workflows/runtime_fuzz_tests.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: facebook/react/fuzz_tests
1+
name: (Runtime) Fuzz tests
22
on:
33
schedule:
44
- cron: 0 * * * *
@@ -28,5 +28,5 @@ jobs:
2828
shell: bash
2929
- name: Run fuzz tests
3030
run: |-
31-
FUZZ_TEST_SEED=$RANDOM yarn test fuzz --ci
32-
FUZZ_TEST_SEED=$RANDOM yarn test --prod fuzz --ci
31+
FUZZ_TEST_SEED=$RANDOM yarn test fuzz --ci=github
32+
FUZZ_TEST_SEED=$RANDOM yarn test --prod fuzz --ci=github

0 commit comments

Comments
 (0)