Skip to content

Commit

Permalink
Revert "[windows] dump_syms - Support ARM64 PDB"
Browse files Browse the repository at this point in the history
This reverts commit e92bea3.

Reason for revert: Causing crashes in symupload.exe for arm64 binaries.

Original change's description:
> [windows] dump_syms - Support ARM64 PDB
>
> Change-Id: I6a9cb5c5253abcd34cb72a035d280b7fd8a50372
> Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5836085
> Reviewed-by: Nelson Billing <[email protected]>

Change-Id: I32c96088213c1196f506580dbcb0b79dae945398
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/5980968
Reviewed-by: Will Harris <[email protected]>
  • Loading branch information
Nelson Billing committed Oct 30, 2024
1 parent f8d05a9 commit 47f7823
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion src/common/windows/pdb_source_line_writer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1050,7 +1050,7 @@ bool PDBSourceLineWriter::PrintFrameDataUsingEXE() {

bool PDBSourceLineWriter::PrintFrameData() {
PDBModuleInfo info;
if (GetModuleInfo(&info) && (info.cpu == L"x86_64" || info.cpu == L"arm64")) {
if (GetModuleInfo(&info) && info.cpu == L"x86_64") {
return PrintFrameDataUsingEXE();
}
return PrintFrameDataUsingPDB();
Expand Down
3 changes: 0 additions & 3 deletions src/common/windows/pe_util.h
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,6 @@ constexpr const wchar_t* FileHeaderMachineToCpuString(WORD machine) {
case IMAGE_FILE_MACHINE_AMD64: {
return L"x86_64";
}
case IMAGE_FILE_MACHINE_ARM64: {
return L"arm64";
}
default: { return L"unknown"; }
}
}
Expand Down

0 comments on commit 47f7823

Please sign in to comment.