[compiler] Add context callee import if required
Previously the compiler would add an import for the specified context callee even if the context access was not lowered, leading to unused imports. This PR tracks if lowering has happened and adds the import only when necessary. ghstack-source-id: 6ad794da41116e1034783b6c4a58fbfe7790343e Pull Request resolved: https://github.com/facebook/react/pull/30628
Showing
- compiler/packages/babel-plugin-react-compiler/src/Entrypoint/Program.ts 4 additions, 1 deletion...ges/babel-plugin-react-compiler/src/Entrypoint/Program.ts
- compiler/packages/babel-plugin-react-compiler/src/HIR/Environment.ts 2 additions, 0 deletions...ckages/babel-plugin-react-compiler/src/HIR/Environment.ts
- compiler/packages/babel-plugin-react-compiler/src/Optimization/LowerContextAccess.ts 2 additions, 1 deletion...gin-react-compiler/src/Optimization/LowerContextAccess.ts
- compiler/packages/babel-plugin-react-compiler/src/ReactiveScopes/CodegenReactiveFunction.ts 6 additions, 0 deletions...ct-compiler/src/ReactiveScopes/CodegenReactiveFunction.ts
- compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/todo.lower-context-access-array-destructuring.expect.md 0 additions, 1 deletion...r/todo.lower-context-access-array-destructuring.expect.md
- compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/todo.lower-context-access-destructure-multiple.expect.md 0 additions, 1 deletion.../todo.lower-context-access-destructure-multiple.expect.md
- compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/todo.lower-context-access-mixed-array-obj.expect.md 0 additions, 1 deletion...piler/todo.lower-context-access-mixed-array-obj.expect.md
- compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/todo.lower-context-access-nested-destructuring.expect.md 0 additions, 1 deletion.../todo.lower-context-access-nested-destructuring.expect.md
- compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/todo.lower-context-access-property-load.expect.md 0 additions, 1 deletion...ompiler/todo.lower-context-access-property-load.expect.md
Please register or sign in to comment