Closed Bug 1271565 Opened 4 years ago Closed 4 years ago
Baseline JIT: 'illegal instruction' crash on aarch64 target
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.75 Safari/537.36 Steps to reproduce: a) modify code to enable baseline JIT on aarch64 b) js-engine standalone compile c) run jsshell with empty loop Actual results: jsshell crashed Expected results: script normally exit
Hardware: Unspecified → Other
Summary: Baseline JIT on arm64 "illeagal instruction" → Baseline JIT: 'illeagal instruction' crash on aarch64 target
Version: 46 Branch → Trunk
Jan, could you figure out what to do with this? Does it need to be a sec bug, etc.? Thanks.
I don't think this has to be s-s.
Summary: Baseline JIT: 'illeagal instruction' crash on aarch64 target → Baseline JIT: 'illegal instruction' crash on aarch64 target
Attachment #8759535 - Flags: review?(sstangl) → review+
This doesn't compile. https://treeherder.mozilla.org/logviewer.html#?job_id=23466407&repo=try#L923
Sorry for my carelessness.Here is a new patch.
Attachment #8768651 - Flags: review?(sstangl) → review+
Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5db2b6f8c1a8 sk, can you please add commit information to your patches so they're easier for people to checkin? Thanks! https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F
Assignee: nobody → sk.alvin.x
(In reply to Ryan VanderMeulen [:RyanVM] from comment #6) > Try push: > https://treeherder.mozilla.org/#/jobs?repo=try&revision=5db2b6f8c1a8 > > sk, can you please add commit information to your patches so they're easier > for people to checkin? Thanks! > https://developer.mozilla.org/en-US/docs/ > Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check- > in_for_me.3F Create a new patch and request for review?
You don't need to re-request review, just uploading a new patch with the missing information included will suffice.
Attachment #8768651 - Attachment is obsolete: true
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/2763206148aa Add autoflush icache for aarch64 target. r=sstangl
You need to log in before you can comment on or make changes to this bug.