This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Remove a redundant function fixupBTI
ClosedPublic

Authored by JiruiWu on Mar 1 2023, 2:02 AM.

Details

Summary

Since the redundant BTI instructions emitted by jump tables are now
removed in the ARMBranchTargets pass, the fixupBTI function is not needed
in the ARMConstantIslandPass. Some related tests are removed as well.

The relevant patch that removes the redundant BTI instructions:
https://reviews.llvm.org/D144470

Diff Detail

Event Timeline

JiruiWu created this revision.Mar 1 2023, 2:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 2:02 AM
JiruiWu requested review of this revision.Mar 1 2023, 2:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 2:02 AM
olista01 accepted this revision.Mar 1 2023, 3:27 AM

LGTM

This revision is now accepted and ready to land.Mar 1 2023, 3:27 AM
This revision was landed with ongoing or failed builds.Mar 1 2023, 7:02 AM
This revision was automatically updated to reflect the committed changes.
llvm/lib/Target/ARM/ARMConstantIslandPass.cpp