This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Add a number of methods to the C API
ClosedPublic

Authored by apaszke on Jul 12 2023, 8:20 AM.

Details

Summary

Those include:

  • mlirFuncSetArgAttr
  • mlirOperationSetOperands
  • mlirRegionTakeBody
  • mlirBlockInsertArgument

Diff Detail

Event Timeline

apaszke created this revision.Jul 12 2023, 8:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 12 2023, 8:20 AM
apaszke requested review of this revision.Jul 12 2023, 8:20 AM
apaszke retitled this revision from [MLIR] Add a number of methods to the C API Those include: * mlirFuncSetArgAttr * mlirOperationSetOperands * mlirRegionTakeBody * mlirBlockInsertArgument to [MLIR] Add a number of methods to the C API.Jul 12 2023, 8:23 AM
apaszke edited the summary of this revision. (Show Details)
apaszke edited the summary of this revision. (Show Details)
ftynse accepted this revision.Jul 12 2023, 9:28 AM
ftynse added inline comments.
mlir/include/mlir-c/Dialect/Func.h
32

Please add some documentation.

mlir/lib/CAPI/Dialect/Func.cpp
14

Normally we shouldn't need this (there's no IWYU rule in LLVM) and we are do have this transitively from virtually any MLIR C++ header.

mlir/lib/CAPI/IR/IR.cpp
502

Nit: don't hardcode the number of stack elements in small vectors.

This revision is now accepted and ready to land.Jul 12 2023, 9:28 AM
apaszke updated this revision to Diff 539609.Jul 12 2023, 9:45 AM
apaszke marked 3 inline comments as done.
apaszke edited the summary of this revision. (Show Details)

Addressed review comments

All requested changes have been applied.

jpienaar accepted this revision.Jul 12 2023, 9:56 AM
This revision was automatically updated to reflect the committed changes.