ignore this and arguments when performing TCO on shadowed functions - fixes #1564

src/babel/transformation/transformers/es6/tail-call.js

@@ -44,12 +44,14 @@ var visitor = {
   },
 
   ThisExpression(node, parent, scope, state) {
-    state.needsThis = true;
-    state.thisPaths.push(this);
+    if (!state.isShadowed) {
+      state.needsThis = true;
+      state.thisPaths.push(this);
+    }
   },
 
   ReferencedIdentifier(node, parent, scope, state) {
-    if (node.name === "arguments") {
+    if (node.name === "arguments" && !state.isShadowed) {
       state.needsArguments = true;
       state.argumentsPaths.push(this);
     }
@@ -67,8 +69,9 @@ class TailCallTransformer {
     this.needsThis = false;
     this.thisPaths = [];
 
-    this.ownerId = path.node.id;
-    this.vars    = [];
+    this.isShadowed = path.isArrowFunctionExpression() || path.is("shadow");
+    this.ownerId    = path.node.id;
+    this.vars       = [];
 
     this.scope = scope;
     this.path  = path;

test/core/fixtures/transformation/es6.tail-call/shadow/actual.js

@@ -0,0 +1,8 @@
+(function(){
+  var foo = () => {
+    this;
+    arguments;
+    foo();
+  };
+  foo();
+});

test/core/fixtures/transformation/es6.tail-call/shadow/expected.js

@@ -0,0 +1,13 @@
+"use strict";
+
+(function () {
+  var _this = this,
+      _arguments = arguments;
+
+  var foo = function foo() {
+    _this;
+    _arguments;
+    foo();
+  };
+  foo();
+});