This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Split intrinsics out of LLVMOps.td
ClosedPublic

Authored by ftynse on Apr 7 2022, 8:25 AM.

Details

Summary

Move the operations that correspond to LLVM IR intrinsics in a separate .td
file. This makes it easier to maintain the intrinsics and decreases the compile
time of LLVMDialect.cpp by ~25%.

Depends On D123310

Diff Detail

Event Timeline

ftynse created this revision.Apr 7 2022, 8:25 AM
Herald added a project: Restricted Project. · View Herald Transcript
ftynse requested review of this revision.Apr 7 2022, 8:25 AM
wsmoses accepted this revision.Apr 13 2022, 8:13 AM
wsmoses added inline comments.
mlir/docs/Dialects/LLVM.md
479

This reads weirdly, perhaps:

The MLIR operation system is open, making it....

This revision is now accepted and ready to land.Apr 13 2022, 8:13 AM
ftynse updated this revision to Diff 422584.Apr 13 2022, 11:48 AM

Fix cmake.

jacquesguan accepted this revision.Apr 13 2022, 7:18 PM
This revision was landed with ongoing or failed builds.Apr 14 2022, 4:23 AM
This revision was automatically updated to reflect the committed changes.