Skip to content
This repository has been archived by the owner on Aug 23, 2022. It is now read-only.

Failing test prebuilt_cfg/x86/linux/cfg/hello_world with aarch64 build/host #753

Open
ekilmer opened this issue May 27, 2021 · 0 comments
Open

Comments

@ekilmer
Copy link
Contributor

ekilmer commented May 27, 2021

In testing on cxx-common repo and LLVM 11, I get the following failure on the aarch64 host:

F0527 14:21:29.388495  3685 InstructionLifter.cpp:558] Check failed: word_size == arg_size Expected integer argument to be machine word size (32 bits) but is is 64 instead in instruction at 8048288
*** Check failure stack trace: ***
    @          0x1201888  google::LogMessageFatal::~LogMessageFatal()
    @           0x76ad98  remill::InstructionLifter::LiftRegisterOperand()
    @           0x768cd4  remill::InstructionLifter::LiftIntoBlock()
    @           0x634dcc  mcsema::InstructionLifter::LiftIntoBlock()
    @           0x631958  mcsema::(anonymous namespace)::LiftInstIntoBlock()
    @           0x62e188  mcsema::(anonymous namespace)::LiftInstIntoFunction()
    @           0x62c490  mcsema::(anonymous namespace)::LiftFunction()
    @           0x62a6cc  mcsema::DefineLiftedFunctions()
    @           0x63639c  mcsema::LiftCodeIntoModule()
    @           0x649f3c  main
    @       0x7f8862e720  __libc_start_main
    @           0x5fed28  (unknown)
/__w/_temp/a078273f-363d-440d-ab05-14f82302c858.sh: line 16:  3685 Aborted                 (core dumped) ./install/bin/mcsema-lift-* --arch x86 --os linux --cfg ../tests/test_suite_generator/generated/prebuilt_cfg/x86/linux/cfg/hello_world --output hello_world.bc

The related amd64 passes without failure (not sure if output is correct, though)

Will update with link to logs and remill,anvill dependencies

ekilmer added a commit to lifting-bits/cxx-common that referenced this issue May 27, 2021
rellic:
* Updated to latest `master` commit
* Remove `|| true` to make sure tests stay passing

remill:
* Updated to latest `master` commit
* Updated vcpkg portfile to latest released version 4.0.24
* Allow failure of tests on aarch64. See lifting-bits/remill#518

anvill:
* Updated to latest `master` commit
* Add other submodule for anvill version compatible with mcsema
* Don't install Python3 components since we don't test them

mcsema:
* Updated to latest `master` commit
* Add pinned anvill version submodule
* Add two smoketests
* Allow failure for `x86` smoketest on aarch64 host. See lifting-bits/mcsema#753
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant