This is an archive of the discontinued LLVM Phabricator instance.

ROCm test support for buildbot.
ClosedPublic

Authored by pcf000 on Mar 21 2022, 12:20 PM.

Details

Summary

Run check-mlir with ROCM flags to catch build breaks

Diff Detail

Event Timeline

pcf000 created this revision.Mar 21 2022, 12:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 21 2022, 12:20 PM
pcf000 requested review of this revision.Mar 21 2022, 12:20 PM

Thanks for the patch, Paul!

Could you remove the unrelated formatting changes, please? If you'd like, please feel free to propose a separate NFC patch with these changes.
Everything else looks good.

pcf000 updated this revision to Diff 417339.Mar 22 2022, 10:59 AM

Restoring some trailing whitespace.

Did you do that the other way around? :)
I see only removed trailing white spaces in this patch now.

Directions said to make the commit and do "arc diff". "arc diff" said it no longer assumes HEAD^ so I did "arc diff HEAD^". That came up as a creation and not an update so I did "arc diff HEAD^ --update D122168" and here we are.

Diff 417339 shows the whitespace added. I can't figure out how to display the union of the two diffs.

I'd just make a new clean diff with only related changes and upload it through the Phabricator WebUI. This might be more controllable and straight forward.

pcf000 updated this revision to Diff 417373.Mar 22 2022, 12:54 PM

Removed some spurious whitespace changes (I hope).

This revision is now accepted and ready to land.Mar 22 2022, 1:07 PM
pcf000 updated this revision to Diff 417604.Mar 23 2022, 7:33 AM

Re-upload to pass pre-merge checks.

It appears that I do not have landing permission.

I can commit on your behalf.
But before that, I'll need a password for the new worker. You can either send it to me directly, or I can assign a random one and send it to you.
Please let me know what you prefer.

Password sent to gkistanova@gmail.com from pcf0@icloud.com. Thanks.

This revision was automatically updated to reflect the committed changes.