-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added Pipeline/Transformation from TTIR -> Flatbuffer (#314) #406
Conversation
Interim Update Fixed CMake
ctx->appendDialectRegistry(registry); | ||
|
||
const auto pipeline = | ||
mlir::PassPipelineInfo::lookup("ttir-to-ttnn-backend-pipeline"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we just call createTTIRToTTNNBackendPipeline
directly?
TTIRToTTNNBackendPipelineOptions options;
//.. maybe set options ..
createTTIRToTTNNBackendPipeline(pm, options);
if (mlir::failed(pm.run(moduleOp))) {
throw std::runtime_error("Failed to run pass manager");
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ran it this way in case there was a more involved method to call Pipeline Options. If it's just as simple as editing the struct then that sounds good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned offline, this task will be completed later while I focus on getting E2E working with explorer. Refer to #446 for when that gets completed.
CMake Fix Interim update
Implemented
ttir-to-ttnn-backend-pipeline
andttnn-to-flatbuffer