This is an archive of the discontinued LLVM Phabricator instance.

Drop U6Imm-related definitions
ClosedPublic

Authored by iii on Apr 4 2023, 5:19 AM.

Details

Summary

They are no longer used, which leads to the -Wunused-function warning.

Diff Detail

Event Timeline

iii created this revision.Apr 4 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 4 2023, 5:19 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
iii requested review of this revision.Apr 4 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 4 2023, 5:19 AM

Ah, right. LGTM, but there's also isU6Imm in AsmParser and printU6ImmOperand in SystemZInstPrinter, which should likewise go away now.

uweigand accepted this revision.Apr 4 2023, 5:37 AM
This revision is now accepted and ready to land.Apr 4 2023, 5:37 AM
iii updated this revision to Diff 510760.Apr 4 2023, 5:43 AM
  • Delete other U6Imm-related definitions.
iii retitled this revision from [SystemZ] Drop decodeU6ImmOperand to Drop U6Imm-related definitions.Apr 4 2023, 5:43 AM
iii edited the summary of this revision. (Show Details)

Oh, right. I also found def U6Imm.

uweigand accepted this revision.Apr 4 2023, 5:48 AM

LGTM, thanks!

This revision was landed with ongoing or failed builds.Apr 4 2023, 6:09 AM
This revision was automatically updated to reflect the committed changes.