This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Delete old plugins
ClosedPublic

Authored by jhuber6 on Jan 28 2023, 3:47 PM.

Details

Summary

It's time to remove the old plugins as the next-gen has already been set
to default in LLVM 16.

Diff Detail

Event Timeline

tianshilei1992 created this revision.Jan 28 2023, 3:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2023, 3:47 PM
tianshilei1992 requested review of this revision.Jan 28 2023, 3:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2023, 3:47 PM

optimize run lines

fix test line again

Where's dynamic_cuda and dynamic_hsa. Also why not replace plugins with nextgen instead of keeping the name.

Where's dynamic_cuda and dynamic_hsa. Also why not replace plugins with nextgen instead of keeping the name.

Oh right, the two dynamic libraries have to be moved.

Also, I don't think the nextgen plugins managed the tests. Does this patch retain that code?

Shall we wait to see if people using the nextgen report breakages before deleting the current/previous? I don't think the latest branch has actually shipped to anyone yet

Seems like the schedule is the end of Feb. I'll update the patch properly by then.

Is this ready?

I can commandeer tomorrow

jhuber6 commandeered this revision.Mar 21 2023, 9:28 AM
jhuber6 edited reviewers, added: tianshilei1992; removed: jhuber6.
jhuber6 updated this revision to Diff 507019.Mar 21 2023, 9:28 AM

Hopefully this works.

tianshilei1992 accepted this revision.Apr 22 2023, 12:59 PM

let’s do it!

This revision is now accepted and ready to land.Apr 22 2023, 12:59 PM
jhuber6 updated this revision to Diff 537518.Jul 5 2023, 2:59 PM

Updating, should be good to go once we remove ve and remote in D154548

This revision was automatically updated to reflect the committed changes.
openmp/libomptarget/plugins/ve/CMakeLists.txt