This is an archive of the discontinued LLVM Phabricator instance.

Remove explicit setting of the CFI jumptable section name.
ClosedPublic

Authored by dmikulin on May 7 2018, 11:04 AM.

Details

Summary

Remove explicit setting of the CFI jumptable section name, it does not appear to be needed: jump table sections are created with .cfi.jumptable suffix. With this change each jump table is placed in a separate section, which allows the linker to re-order them.

Diff Detail

Event Timeline

dmikulin created this revision.May 7 2018, 11:04 AM
pcc accepted this revision.May 7 2018, 11:36 AM
pcc added a reviewer: eugenis.

LGTM

llvm/test/Transforms/LowerTypeTests/function-arm-thumb.ll
34

Nit: I'd drop the {{.*}} here and in function.ll, as they don't appear to be needed.

This revision is now accepted and ready to land.May 7 2018, 11:36 AM
dmikulin marked an inline comment as done.May 7 2018, 2:06 PM
This revision was automatically updated to reflect the committed changes.