Page MenuHomePhabricator

[mlir] Remove locking for dialect/operation registration.

Authored by rriddle on Jun 25 2020, 1:35 PM.



Moving forward dialects should only be registered in a thread safe context. This matches the existing usage we have today, but it allows for removing quite a bit of expensive locking from the context.

This led to ~.5 a second compile time improvement when running one conversion pass on a very large .mlir file(hundreds of thousands of operations).

Depends On D81883

Diff Detail

Event Timeline

rriddle created this revision.Jun 25 2020, 1:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 25 2020, 1:35 PM
jpienaar accepted this revision.Jun 28 2020, 7:52 AM
This revision is now accepted and ready to land.Jun 28 2020, 7:52 AM
This revision was automatically updated to reflect the committed changes.