Skip to content

Commit

Permalink
[BUGFIX] Widen assert to 32bits call for AoT
Browse files Browse the repository at this point in the history
  • Loading branch information
chadhietala committed Jan 25, 2019
1 parent c4fbe0a commit a20d5e2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion packages/@glimmer/integration-tests/test/smoke-test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { RenderTest, test, jitSuite } from '@glimmer/integration-tests';
import { RenderTest, test, jitSuite, aotSuite } from '@glimmer/integration-tests';

class SmokeTests extends RenderTest {
static suiteName = 'Smoke Tests';
Expand All @@ -19,3 +19,4 @@ class SmokeTests extends RenderTest {
}

jitSuite(SmokeTests);
aotSuite(SmokeTests);
2 changes: 1 addition & 1 deletion packages/@glimmer/runtime/lib/vm/low-level.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ export default class LowLevelVM {

// Save $pc into $ra, then jump to a new address in `program` (jal in MIPS)
call(handle: number) {
assert(handle < 0b1111111111111111, `Jumping to placehoder address`);
assert(handle < 0xffffffff, `Jumping to placehoder address`);

this.registers[$ra] = this.registers[$pc];
this.setPc(this.heap.getaddr(handle));
Expand Down

0 comments on commit a20d5e2

Please sign in to comment.