HomePhabricator

Fix bug in ArchSpec::MergeFrom
5e35046fb56eUnpublished

Authored by clayborg on May 8 2019, 3:03 PM.

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Fix bug in ArchSpec::MergeFrom

Previous ArchSpec tests didn't catch this bug since we never tested just the OS being out of date. Fixed the bug and covered this with a test that would catch this.

This was found when trying to load a core file where the core file was an ELF file with just the e_machine for architeture and where the ELF header had no OS set in the OSABI field of the e_ident. It wasn't merging the architecture with the target architecture correctly.

Differential Revision: https://reviews.llvm.org/D61659

llvm-svn: 360292

Details

Committed
sidorovdMay 24 2019, 5:38 AM
Differential Revision
D61659: Fix bug in ArchSpec::MergeFrom

Commit No Longer Exists

This commit no longer exists in the repository.