HomePhabricator

Merging r351421:
Audit RequiredrL351452

Description

Merging r351421:

r351421 | ssijaric | 2019-01-17 10:45:17 +0100 (Thu, 17 Jan 2019) | 12 lines

[ARM64][Windows] Share unwind codes between epilogues

There are cases where we have multiple epilogues that have the exact same unwind
code sequence. In that case, the epilogues can share the same unwind codes in
the .xdata section. This should get us past the assert "SEH unwind data
splitting not yet implemented" in many cases.

We still need to add support for generating multiple .pdata/.xdata sections for
those functions that need to be split into fragments.

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


Details

Auditors
grosbach
Committed
hansJan 17 2019, 7:39 AM
Differential Revision
D56813: [AArch64] [Windows] Share unwind codes between epilogues with identical unwind codes
Parents
rL351451: Merging r351370:
Branches
Unknown
Tags
Unknown

Event Timeline

This commit now requires audit.Jan 17 2019, 7:43 AM
RKSimon added inline comments.
/llvm/branches/release_80/lib/MC/MCWin64EH.cpp
520

@hans We're seeing unused variable warnings on NumUnwindCodes - ok to remove?

hans marked an inline comment as done.Jan 18 2019, 3:01 AM
hans added inline comments.
/llvm/branches/release_80/lib/MC/MCWin64EH.cpp
520

I see Ben fixed this on trunk with http://llvm.org/r351426
I've merged that over in r351543.

Thanks for letting me know!