[PATCH] [Object] Search for architecures by name in MachOUniversalBinary::getObjectForArch()
Frederic Riss
friss at apple.com
Mon Jun 22 09:14:34 PDT 2015
Hi samsonov, bogner,
The reason we need to search by name rather than by Triple::ArchType
is to handle x86_64h correclty. There is no different ArchType
for the x86_64h architecture (it identifies itself as x86_64), and
this is by design. The only way to get to the x86_64h slice of an
universal binary is to serch by name.
This issue led to hard to debug and transient symbolication failures
in Asan tests (it mostly works, because the files are very similar).
This also affects the Profiling infrastucture as it is the other user
of that API.
http://reviews.llvm.org/D10604
Files:
include/llvm/Object/MachOUniversal.h
include/llvm/ProfileData/CoverageMapping.h
include/llvm/ProfileData/CoverageMappingReader.h
lib/Object/MachOUniversal.cpp
lib/ProfileData/CoverageMapping.cpp
lib/ProfileData/CoverageMappingReader.cpp
test/tools/llvm-symbolizer/Inputs/fat.x86_64h.c
test/tools/llvm-symbolizer/Inputs/fat.x86_64h.o
test/tools/llvm-symbolizer/fat.test
tools/llvm-cov/CodeCoverage.cpp
tools/llvm-symbolizer/LLVMSymbolize.cpp
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10604.28120.patch
Type: text/x-patch
Size: 7452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150622/2f39a9f8/attachment.bin>
More information about the llvm-commits
mailing list