Fix #4724 (#4737)

The handling of hoisted nodes in class bodies was incorrect, as the node
was being unwrapped *before* checking if it was hoisted, meaning nodes
that should have been hoisted would be output normally.

This affected `PassthroughLiteral`s as they were wrapped in a `Value`.

Author: connec

lib/coffeescript/nodes.js

@@ -497,17 +497,19 @@ exports.Block = class Block extends Base
     compiledNodes = []
 
     for node, index in @expressions
+      if node.hoisted
+        # This is a hoisted expression.
+        # We want to compile this and ignore the result.
+        node.compileToFragments o
+        continue
+
       node = node.unwrapAll()
       node = (node.unfoldSoak(o) or node)
       if node instanceof Block
         # This is a nested block. We don’t do anything special here like
         # enclose it in a new scope; we just compile the statements in this
         # block along with our own.
         compiledNodes.push node.compileNode o
-      else if node.hoisted
-        # This is a hoisted expression.
-        # We want to compile this and ignore the result.
-        node.compileToFragments o
       else if top
         node.front = yes
         fragments = node.compileToFragments o

src/nodes.coffee

@@ -1848,3 +1848,12 @@ test "#4464: backticked expressions in class body", ->
   b = new B
   eq 42, b.x
   eq 84, b.y
+
+test "#4724: backticked expression in a class body with hoisted member", ->
+  class A
+    `get x() { return 42; }`
+    hoisted: 84
+
+  a = new A
+  eq 42, a.x
+  eq 84, a.hoisted