More asserts

Common/GPU/Vulkan/VulkanRenderManager.cpp

@@ -422,6 +422,14 @@ void VulkanRenderManager::DestroyBackbuffers() {
 	queueRunner_.DestroyBackBuffers();
 }
 
+void VulkanRenderManager::CheckNothingPending() {
+	_assert_(pipelinesToCheck_.empty());
+	{
+		std::unique_lock<std::mutex> lock(compileMutex_);
+		_assert_(compileQueue_.empty());
+	}
+}
+
 VulkanRenderManager::~VulkanRenderManager() {
 	INFO_LOG(G3D, "VulkanRenderManager destructor");
 

Common/GPU/Vulkan/VulkanRenderManager.h

@@ -115,13 +115,12 @@ class VKRGraphicsPipelineDesc : public Draw::RefCountedObject {
 	RPKey rpKey{};
 };
 
-// Wrapped pipeline. Doesn't own desc.
+// Wrapped pipeline. Does own desc!
 struct VKRGraphicsPipeline {
 	VKRGraphicsPipeline(PipelineFlags flags, const char *tag) : flags_(flags), tag_(tag) {}
 	~VKRGraphicsPipeline();
 
 	bool Create(VulkanContext *vulkan, VkRenderPass compatibleRenderPass, RenderPassType rpType, VkSampleCountFlagBits sampleCount, double scheduleTime, int countToCompile);
-
 	void DestroyVariants(VulkanContext *vulkan, bool msaaOnly);
 
 	// This deletes the whole VKRGraphicsPipeline, you must remove your last pointer to it when doing this.
@@ -237,6 +236,7 @@ class VulkanRenderManager {
 	// These can run on a different thread!
 	void Finish();
 	void Present();
+	void CheckNothingPending();
 
 	void SetInvalidationCallback(InvalidationCallback callback) {
 		invalidationCallback_ = callback;

GPU/Vulkan/GPU_Vulkan.cpp

@@ -169,6 +169,7 @@ GPU_Vulkan::~GPU_Vulkan() {
 		VulkanRenderManager *rm = (VulkanRenderManager *)draw_->GetNativeObject(Draw::NativeObject::RENDER_MANAGER);
 		// This now also does a hard sync with the render thread, so that we can safely delete our pipeline layout below.
 		rm->StopThreads();
+		rm->CheckNothingPending();
 	}
 
 	SaveCache(shaderCachePath_);

android/src/org/ppsspp/ppsspp/NativeActivity.java

@@ -820,6 +820,7 @@ protected void onDestroy() {
 		if (isVRDevice()) {
 			System.exit(0);
 		}
+		Log.i(TAG, "onDestroy");
 	}
 
 	@Override