This is an archive of the discontinued LLVM Phabricator instance.

Remove TODO related to adding assert from Sparse Tensor Pipeline code
ClosedPublic

Authored by vmrajas on Aug 25 2022, 12:25 PM.

Details

Summary

Removing the TODO related to asserting that original pm is for ModuleOp.
The TODO is removed for following reasons:

  1. There is no easy way to do this. We currently don't have this information stored in OpPassManager object.
  2. There are currently no consumers of this information and storing this information with OpPassManager for a simple assert will be an overkill.

Diff Detail

Event Timeline

vmrajas created this revision.Aug 25 2022, 12:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 25 2022, 12:25 PM
vmrajas requested review of this revision.Aug 25 2022, 12:25 PM
aartbik accepted this revision.Aug 25 2022, 1:09 PM
This revision is now accepted and ready to land.Aug 25 2022, 1:09 PM