-
Notifications
You must be signed in to change notification settings - Fork 517
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
Move CIs to use docker builds #1316
Conversation
Now that #1234 has landed and anyone can run CI / Release builds locally move GHA to use the same flow.
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.
LGTM with some nit requests.
@silvasean if you have a quick fix for #1334 lets land that and then this change. If not we can revert the last shape lib test and re-land after this lands. I would like to try and land this today so I can tend to the CI over the long weekend in case anything needs tweaking / reverts etc. |
Let's land this! The fix is in #1338 -- feel free to merge it yourself if needed. |
awesome thanks. will wait for it to go green and land #1338 and then land this when green. |
Awesome. Thanks Anush for all the hard work here and sorry for these last minute speed bumps on the generated files! |
Signed-off-by: Whitney Tsang <[email protected]>
The modification to use the builder based interface to generate Krnl loop is completed (llvm#1250, llvm#1283, llvm#1285, llvm#1292, llvm#1293, llvm#1303, llvm#1308, llvm#1314, llvm#1316, llvm#1317, llvm#1326, llvm#1403), and BuildKrnlLoop is no longer needed. Signed-off-by: Whitney Tsang [email protected]
Now that #1234 has landed and anyone can run CI / Release builds locally move GHA to use the same flow.