This is an archive of the discontinued LLVM Phabricator instance.

[MC] Add section flag 'l' for SHF_X86_64_LARGE
ClosedPublic

Authored by aeubanks on Apr 14 2023, 11:06 AM.

Diff Detail

Event Timeline

aeubanks created this revision.Apr 14 2023, 11:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2023, 11:06 AM
aeubanks requested review of this revision.Apr 14 2023, 11:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2023, 11:06 AM

This should be disabled for non-x86-64 triples.

This should be disabled for non-x86-64 triples.

if you look above my change, there are other arch-specific flags that aren't gated behind the assembly arch, this is consistent with that

This should be disabled for non-x86-64 triples.

if you look above my change, there are other arch-specific flags that aren't gated behind the assembly arch, this is consistent with that

I'm working on a patch to add errors...

aeubanks updated this revision to Diff 513820.Apr 14 2023, 5:20 PM

rebase, don't allow for for non-x86_64

MaskRay accepted this revision.Apr 14 2023, 5:25 PM
This revision is now accepted and ready to land.Apr 14 2023, 5:25 PM
MaskRay added inline comments.Apr 14 2023, 5:25 PM
llvm/test/MC/ELF/section-flags-unknown.s
1

This will need a REQUIRES:.

This revision was landed with ongoing or failed builds.Apr 14 2023, 5:32 PM
This revision was automatically updated to reflect the committed changes.