Assume that accessing members of a pure global function or its prototype

is never a side-effect.

src/ast/nodes/CallExpression.js

@@ -1,6 +1,5 @@
 import Node from '../Node.js';
 import CallOptions from '../CallOptions';
-import ExecutionPathOptions from '../ExecutionPathOptions';
 
 export default class CallExpression extends Node {
 	bindAssignmentAtPath ( path, expression, options ) {

src/ast/nodes/NewExpression.js

@@ -1,6 +1,5 @@
 import Node from '../Node.js';
 import CallOptions from '../CallOptions';
-import ExecutionPathOptions from '../ExecutionPathOptions';
 
 export default class NewExpression extends Node {
 	hasEffects ( options ) {

src/ast/variables/GlobalVariable.js

@@ -17,7 +17,11 @@ export default class GlobalVariable extends Variable {
 	hasEffectsWhenAccessedAtPath ( path ) {
 		// path.length == 0 can also have an effect but we postpone this for now
 		return path.length > 0
-			&& !pureFunctions[ [ this.name, ...path ].join( '.' ) ];
+			&& !pureFunctions[ [ this.name, ...path ].join( '.' ) ]
+			&& !pureFunctions[ [ this.name, ...path.slice( 0, -1 ) ].join( '.' ) ]
+			&& !(path.length > 1
+				&& pureFunctions[ [ this.name, ...path.slice( 0, -2 ) ].join( '.' ) ]
+				&& path[ path.length - 2 ] === 'prototype');
 	}
 
 	hasEffectsWhenCalledAtPath ( path ) {

test/form/samples/globals-removes-access-to-pure-function-members/_config.js

@@ -0,0 +1,5 @@
+var path = require('path');
+
+module.exports = {
+	description: 'accessing members of pure functions and their prototypes is not a side-effect'
+};

test/form/samples/globals-removes-access-to-pure-function-members/_expected/amd.js

@@ -0,0 +1,6 @@
+define(function () { 'use strict';
+
+	Unknown.staticMember;
+	Unknown.prototype.method;
+
+});

test/form/samples/globals-removes-access-to-pure-function-members/_expected/cjs.js

@@ -0,0 +1,4 @@
+'use strict';
+
+Unknown.staticMember;
+Unknown.prototype.method;

test/form/samples/globals-removes-access-to-pure-function-members/_expected/es.js

@@ -0,0 +1,2 @@
+Unknown.staticMember;
+Unknown.prototype.method;

test/form/samples/globals-removes-access-to-pure-function-members/_expected/iife.js

@@ -0,0 +1,7 @@
+(function () {
+	'use strict';
+
+	Unknown.staticMember;
+	Unknown.prototype.method;
+
+}());

test/form/samples/globals-removes-access-to-pure-function-members/_expected/umd.js

@@ -0,0 +1,10 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory() :
+	typeof define === 'function' && define.amd ? define(factory) :
+	(factory());
+}(this, (function () { 'use strict';
+
+	Unknown.staticMember;
+	Unknown.prototype.method;
+
+})));

test/form/samples/globals-removes-access-to-pure-function-members/main.js

@@ -0,0 +1,6 @@
+Symbol.iterator;
+Array.prototype.reverse;
+Object.prototype.hasOwnProperty;
+
+Unknown.staticMember;
+Unknown.prototype.method;

test/form/samples/removes-unused-babel-helpers/_config.js

@@ -0,0 +1,5 @@
+const assert = require( 'assert' );
+
+module.exports = {
+	description: 'Removes unused babel helpers from the build (#1595)'
+};

test/form/samples/removes-unused-babel-helpers/_expected/amd.js

@@ -0,0 +1,5 @@
+define(function () { 'use strict';
+
+
+
+});

test/form/samples/removes-unused-babel-helpers/_expected/cjs.js

@@ -0,0 +1,2 @@
+'use strict';
+

test/form/samples/removes-unused-babel-helpers/_expected/es.js

@@ -0,0 +1 @@
+

test/form/samples/removes-unused-babel-helpers/_expected/iife.js

@@ -0,0 +1,6 @@
+(function () {
+	'use strict';
+
+
+
+}());

test/form/samples/removes-unused-babel-helpers/_expected/umd.js

@@ -0,0 +1,9 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory() :
+	typeof define === 'function' && define.amd ? define(factory) :
+	(factory());
+}(this, (function () { 'use strict';
+
+
+
+})));