From 554890cda88eea2053a1fc92bf5060e1df0c26c8 Mon Sep 17 00:00:00 2001
From: nihuini <nihuini@tencent.com>
Date: Fri, 12 Jun 2020 10:17:32 +0800
Subject: [PATCH] fp16p and fp16s cannot be both enabled in shader source

---
 src/gpu.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/gpu.cpp b/src/gpu.cpp
index 04cfc3a6d..79bc35def 100644
--- a/src/gpu.cpp
+++ b/src/gpu.cpp
@@ -2722,14 +2722,15 @@ int compile_spirv_module(int shader_type_index, const Option& opt, std::vector<u
 
     custom_defines.push_back(std::make_pair("psc(x)", "(x==0?p.x:x)"));
 
-    if (opt.use_fp16_packed)
-    {
-        custom_defines.push_back(std::make_pair("NCNN_fp16_packed", "1"));
-    }
     if (opt.use_fp16_storage)
     {
         custom_defines.push_back(std::make_pair("NCNN_fp16_storage", "1"));
     }
+    else if (opt.use_fp16_packed)
+    {
+        custom_defines.push_back(std::make_pair("NCNN_fp16_packed", "1"));
+    }
+
     if (opt.use_fp16_arithmetic)
     {
         custom_defines.push_back(std::make_pair("NCNN_fp16_arithmetic", "1"));