This is an archive of the discontinued LLVM Phabricator instance.

[mlir][Vector] add pass option to `convert-vector-to-llvm` to emit opaque pointers
ClosedPublic

Authored by zero9178 on Feb 12 2023, 12:07 PM.

Details

Summary

Part of https://discourse.llvm.org/t/rfc-switching-the-llvm-dialect-and-dialect-lowerings-to-opaque-pointers/68179

This patch allows the use of convert-vector-to-llvm with LLVM opaque pointer types. The changes required were relatively minor:

  • there were a few GEPs and loads that required specifying the result or base type
  • a few bitcasts, especially when interacting with memref have to not be emitted when using opaque pointers

Diff Detail

Event Timeline

zero9178 created this revision.Feb 12 2023, 12:07 PM
zero9178 requested review of this revision.Feb 12 2023, 12:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2023, 12:07 PM
zero9178 retitled this revision from [mlir][Vecotr] add pass option to `convert-vector-to-llvm` to emit opaque pointers to [mlir][Vector] add pass option to `convert-vector-to-llvm` to emit opaque pointers.Feb 12 2023, 12:08 PM
gysit accepted this revision.Feb 12 2023, 11:10 PM

LGTM!

This revision is now accepted and ready to land.Feb 12 2023, 11:10 PM