Differential D135639

[MC] Consider IsMTETaggedFrame in CIEKey
ClosedPublic
Actions

Authored by fmayer on Oct 10 2022, 7:49 PM.

Details

Reviewers

eugenis

Commits

rG42cdec1134cb: [MC] Consider IsMTETaggedFrame in CIEKey

Summary

Before this, we would incorrectly coalesce CIE for frames with and
without stack MTE.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

fmayer created this revision.Oct 10 2022, 7:49 PM

Herald added a project: Restricted Project. · View Herald TranscriptOct 10 2022, 7:49 PM

Herald added subscribers: mikhail.ramalho, hiraditya. · View Herald Transcript

Harbormaster completed remote builds in B191416: Diff 466689.Oct 10 2022, 8:29 PM

test

Herald added a subscriber: emaste. · View Herald TranscriptOct 11 2022, 1:03 PM

fmayer added a parent revision: D135638: [MC] Also sort on IsBKeyFrame.Oct 11 2022, 1:03 PM

fmayer added a reviewer: pcc.

Harbormaster completed remote builds in B191560: Diff 466896.Oct 11 2022, 2:28 PM

fmayer published this revision for review.Oct 12 2022, 12:44 PM

fmayer edited reviewers, added: eugenis; removed: pcc.

Herald added a project: Restricted Project. · View Herald TranscriptOct 12 2022, 12:44 PM

Herald added subscribers: llvm-commits, MaskRay. · View Herald Transcript

LGTM

This revision is now accepted and ready to land.Oct 12 2022, 12:52 PM

messed up rebase before

This revision was landed with ongoing or failed builds.Oct 12 2022, 2:51 PM

Closed by commit rG42cdec1134cb: [MC] Consider IsMTETaggedFrame in CIEKey (authored by fmayer). · Explain Why

This revision was automatically updated to reflect the committed changes.

fmayer added a commit: rG42cdec1134cb: [MC] Consider IsMTETaggedFrame in CIEKey.

Harbormaster completed remote builds in B191814: Diff 467257.Oct 12 2022, 3:00 PM

Revision Contents
Changeset List

Path

Size

llvm/

lib/

MC/

MCDwarf.cpp

29 lines

test/

MC/

ELF/

AArch64/

cfi.s

3 lines

Diff 467270

View Options

llvm/lib/MC/MCDwarf.cpp