Hermes currently has divergent function hoisting behavior compared to QuickJS, V8, and JSC in non-strict mode. The following code prints Error1; Error1 instead of Error; Error1

```
{
    function test1(a) {
        if(a == "a") {
            test1("b");
            return;
        }
        print ('Error');
    }

    callback1 = test1;
}

{
    function test1(a) {
        if(a == "a") {
            test1("b");
            return;
        }
        print ('Error1');
    }

    callback2 = test1;
}

callback1("a");
callback2("a");
```

This happens as scoped function promoted to global scope are treated as global function, like for `var`.
New approach:
- Creating two declaration associated with each promoted function identifier and storing the function code into two pointers, one global and one scoped. Each declaration has one pointer associated.
- Each identifier which is not a declaration is resolved into either the global or scoped declaration of the promoted function

Test 262 results:
- 100% on test/statements, 98.53% overall. **Nothing changed in the results** before and after the change.

---

…ting