Merge pull request #8086 from webpack/bugfix/comments-concat-export-d…

…efault

fixes #8085

lib/optimize/ConcatenatedModule.js

@@ -1364,12 +1364,20 @@ class HarmonyExportExpressionDependencyConcatenatedTemplate {
 		}
 
 		if (dep.range) {
-			source.replace(dep.rangeStatement[0], dep.range[0] - 1, content + "(");
+			source.replace(
+				dep.rangeStatement[0],
+				dep.range[0] - 1,
+				content + "(" + dep.prefix
+			);
 			source.replace(dep.range[1], dep.rangeStatement[1] - 1, ");");
 			return;
 		}
 
-		source.replace(dep.rangeStatement[0], dep.rangeStatement[1] - 1, content);
+		source.replace(
+			dep.rangeStatement[0],
+			dep.rangeStatement[1] - 1,
+			content + dep.prefix
+		);
 	}
 }
 

test/configCases/scope-hoisting/harmony-pure-default/index.js

@@ -0,0 +1,15 @@
+import x1, { value as v1 } from "./module1";
+import x2, { value as v2 } from "./module2";
+import { value as v3 } from "./module3";
+import x4, { value as v4 } from "./module4";
+
+it("should not execute exports when annotated with pure comment", () => {
+	expect(v1).toBe(42);
+	expect(v2).toBe(42);
+	expect(v3).toBe(42);
+	expect(v4).toBe(42);
+});
+
+var x = /*#__PURE__*/(function() {
+	return x1 + x2 + x4;
+});

test/configCases/scope-hoisting/harmony-pure-default/module1.js

@@ -0,0 +1,9 @@
+let value = 42;
+
+const inc = () => {
+	value++;
+};
+
+export default /*#__PURE__*/inc();
+
+export { value };

test/configCases/scope-hoisting/harmony-pure-default/module2.js

@@ -0,0 +1,9 @@
+let value = 42;
+
+const inc = () => {
+	value++;
+};
+
+export default (/*#__PURE__*/inc());
+
+export { value };

test/configCases/scope-hoisting/harmony-pure-default/module3.js

@@ -0,0 +1,9 @@
+let value = 42;
+
+const inc = () => {
+	value++;
+};
+
+export default /*#__PURE__*/(inc());
+
+export { value };

test/configCases/scope-hoisting/harmony-pure-default/module4.js

@@ -0,0 +1,15 @@
+let value = 42;
+
+const inc = () => {
+	value++;
+};
+
+export
+// hello
+default
+// world
+/*#__PURE__*/
+inc()
+;
+
+export { value };

test/configCases/scope-hoisting/harmony-pure-default/webpack.config.js

@@ -0,0 +1,7 @@
+module.exports = {
+	mode: "production",
+	optimization: {
+		minimize: true,
+		concatenateModules: true
+	}
+};