This is an archive of the discontinued LLVM Phabricator instance.

[libunwind] [SEH] Allow setting/getting the register UNW_X86_64_RIP
ClosedPublic

Authored by mstorsjo on Apr 5 2023, 9:58 AM.

Details

Summary

This fixes libunwind_01.pass.cpp for x86_64 Windows.

Diff Detail

Event Timeline

mstorsjo created this revision.Apr 5 2023, 9:58 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptApr 5 2023, 9:58 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a subscriber: pengfei. · View Herald Transcript
mstorsjo requested review of this revision.Apr 5 2023, 9:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2023, 9:58 AM
cdavis5x accepted this revision.Apr 6 2023, 1:31 PM

Huh. I must've thought the specific number for rip would never have been passed.

Can someone give a libunwind group approval for this one? This one is quite trivial.

MaskRay accepted this revision.Apr 12 2023, 7:34 PM
MaskRay added a subscriber: MaskRay.

Looks great!

ELF platforms use Registers_x86_64::validRegister which returns true for a reg number within [0,16].

This revision is now accepted and ready to land.Apr 12 2023, 7:34 PM
This revision was landed with ongoing or failed builds.Apr 13 2023, 2:06 AM
This revision was automatically updated to reflect the committed changes.