This is an archive of the discontinued LLVM Phabricator instance.

Remove uses of llvm_shutdown
Needs ReviewPublic

Authored by nhaehnle on Jul 5 2022, 2:20 AM.

Details

Summary

With the removal of ManagedStatic, llvm_shutdown will be removed as
well.

Diff Detail

Event Timeline

nhaehnle created this revision.Jul 5 2022, 2:20 AM
Herald added a reviewer: rafauler. · View Herald Transcript
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a reviewer: maksfb. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
nhaehnle requested review of this revision.Jul 5 2022, 2:20 AM
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptJul 5 2022, 2:20 AM
yota9 removed a subscriber: yota9.Jul 5 2022, 2:25 AM
sunho added a subscriber: sunho.Jul 10 2022, 7:29 AM
Meinersbur added inline comments.
polly/lib/External/isl/interface/extract_interface.cc
590

This file is imported from the upstream project (https://repo.or.cz/isl.git/blob/295cf91923295ca694e9e4433a0b818150421bee:/interface/extract_interface.cc#l590) and this change will be lost when synchronizing with it. However, this file is also not used within LLVM. I recommend to just keep as-is.

nhaehnle added inline comments.Aug 3 2022, 2:36 AM
polly/lib/External/isl/interface/extract_interface.cc
590

Thanks, will do.

nhaehnle updated this revision to Diff 449609.Aug 3 2022, 2:36 AM

Address review comment

efriedma added inline comments.Aug 3 2022, 11:16 AM
polly/lib/External/isl/interface/extract_interface.cc
590

I think you missed the other change in this file?