Merge branch 'master' of https://github.com/rollup/rollup into sync-b379a592

Author: docschina-bot

.github/workflows/build-and-tests.yml

@@ -181,6 +181,16 @@ jobs:
               rustup target add riscv64gc-unknown-linux-gnu &&
               npm run build:napi -- --release --target riscv64gc-unknown-linux-gnu &&
               riscv64-linux-gnu-strip *.node
+          - host: ubuntu-latest
+            target: powerpc64le-unknown-linux-gnu
+            setup: |
+              sudo apt-get update
+              sudo apt-get install gcc-powerpc64le-linux-gnu -y
+            build: >-
+              set -e &&
+              export CARGO_TARGET_POWERPC64LE_UNKNOWN_LINUX_GNU_LINKER=powerpc64le-linux-gnu-gcc && rustup target add powerpc64le-unknown-linux-gnu &&
+              npm run build:napi -- --release --target powerpc64le-unknown-linux-gnu &&
+              powerpc64le-linux-gnu-strip *.node
           - host: ubuntu-latest
             target: s390x-unknown-linux-gnu
             setup: |

CHANGELOG.md

@@ -1,5 +1,19 @@
 # rollup changelog
 
+## 4.13.2
+
+_2024-03-28_
+
+### Bug Fixes
+
+- Ensure accessing module info is cached after the build phase for improved performance (#5438)
+- Support powerpc64le CPUs (#5350)
+
+### Pull Requests
+
+- [#5350](https://github.com/rollup/rollup/pull/5350): Add support for ppc64le (@pavolloffay, @lukastaegert)
+- [#5438](https://github.com/rollup/rollup/pull/5438): Cache module info getters before output generation (@bluwy, @lukastaegert)
+
 ## 4.13.1
 
 _2024-03-27_

browser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rollup/browser",
-  "version": "4.13.1",
+  "version": "4.13.2",
   "description": "Next-generation ES module bundler browser build",
   "main": "dist/rollup.browser.js",
   "module": "dist/es/rollup.browser.js",

native.js

@@ -16,6 +16,7 @@ const bindingsByPlatformAndArch = {
 	linux: {
 		arm: { base: 'linux-arm-gnueabihf', musl: null },
 		arm64: { base: 'linux-arm64-gnu', musl: 'linux-arm64-musl' },
+		ppc64le: { base: 'linux-powerpc64le-gnu', musl: null },
 		riscv64: { base: 'linux-riscv64-gnu', musl: null },
 		s390x: { base: 'linux-s390x-gnu', musl: null },
 		x64: { base: 'linux-x64-gnu', musl: 'linux-x64-musl' }

npm/linux-powerpc64le-gnu/README.md

@@ -0,0 +1,3 @@
+# `@rollup/rollup-linux-powerpc64le-gnu`
+
+This is the **powerpc64le-unknown-linux-gnu** binary for `rollup`

npm/linux-powerpc64le-gnu/package.json

@@ -0,0 +1,22 @@
+{
+  "name": "@rollup/rollup-linux-powerpc64le-gnu",
+  "version": "0.0.0",
+  "os": [
+    "linux"
+  ],
+  "cpu": [
+    "ppc64le"
+  ],
+  "files": [
+    "rollup.linux-powerpc64le-gnu.node"
+  ],
+  "description": "Native bindings for Rollup",
+  "author": "Lukas Taegert-Atkinson",
+  "homepage": "https://rollupjs.org/",
+  "license": "MIT",
+  "repository": "rollup/rollup",
+  "libc": [
+    "glibc"
+  ],
+  "main": "./rollup.linux-powerpc64le-gnu.node"
+}

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "rollup",
-  "version": "4.13.1",
+  "version": "4.13.2",
   "description": "Next-generation ES module bundler",
   "main": "dist/rollup.js",
   "module": "dist/es/rollup.js",
@@ -25,6 +25,7 @@
         "armv7-unknown-linux-gnueabihf",
         "i686-pc-windows-msvc",
         "riscv64gc-unknown-linux-gnu",
+        "powerpc64le-unknown-linux-gnu",
         "s390x-unknown-linux-gnu",
         "x86_64-apple-darwin",
         "x86_64-pc-windows-msvc",

package.json

@@ -1,6 +1,7 @@
 import ExternalVariable from './ast/variables/ExternalVariable';
 import type { CustomPluginOptions, ModuleInfo, NormalizedInputOptions } from './rollup/types';
 import { EMPTY_ARRAY } from './utils/blank';
+import { cacheObjectGetters } from './utils/getter';
 import { makeLegal } from './utils/identifierHelpers';
 import { LOGLEVEL_WARN } from './utils/logging';
 import { logUnusedExternalImports } from './utils/logs';
@@ -59,6 +60,10 @@ export default class ExternalModule {
 		};
 	}
 
+	cacheInfoGetters(): void {
+		cacheObjectGetters(this.info, ['dynamicImporters', 'importers']);
+	}
+
 	getVariableForExportName(name: string): [variable: ExternalVariable] {
 		const declaration = this.declarations.get(name);
 		if (declaration) return [declaration];

src/ExternalModule.ts

@@ -151,6 +151,7 @@ export default class Graph {
 			throw new Error('You must supply options.input to rollup');
 		}
 		for (const module of this.modulesById.values()) {
+			module.cacheInfoGetters();
 			if (module instanceof Module) {
 				this.modules.push(module);
 			} else {

src/Graph.ts

@@ -55,6 +55,7 @@ import { decodedSourcemap, resetSourcemapCache } from './utils/decodedSourcemap'
 import { getId } from './utils/getId';
 import { getNewSet, getOrCreate } from './utils/getOrCreate';
 import { getOriginalLocation } from './utils/getOriginalLocation';
+import { cacheObjectGetters } from './utils/getter';
 import { makeLegal } from './utils/identifierHelpers';
 import { LOGLEVEL_WARN } from './utils/logging';
 import {
@@ -390,6 +391,22 @@ export default class Module {
 		this.ast!.bind();
 	}
 
+	cacheInfoGetters(): void {
+		cacheObjectGetters(this.info, [
+			'dynamicallyImportedIdResolutions',
+			'dynamicallyImportedIds',
+			'dynamicImporters',
+			'exportedBindings',
+			'exports',
+			'hasDefaultExport',
+			'implicitlyLoadedAfterOneOf',
+			'implicitlyLoadedBefore',
+			'importedIdResolutions',
+			'importedIds',
+			'importers'
+		]);
+	}
+
 	error(properties: RollupError, pos: number | undefined): never {
 		pos !== undefined && this.addLocationToLogProps(properties, pos);
 		return error(properties);

src/Module.ts

@@ -0,0 +1,16 @@
+export function cacheObjectGetters<T, K extends PropertyKey = keyof T>(
+	object: T,
+	getterProperties: K[]
+) {
+	for (const property of getterProperties) {
+		const propertyGetter = Object.getOwnPropertyDescriptor(object, property)!.get!;
+		Object.defineProperty(object, property, {
+			get() {
+				const value = propertyGetter.call(object);
+				// This replaces the getter with a fixed value for subsequent calls
+				Object.defineProperty(object, property, { value });
+				return value;
+			}
+		});
+	}
+}