[SPIR-V] Add CapabilityFPGABufferLocationINTEL only when the decoration is added (#2240)

MrSidims · web-flow · commit 88cbe10e94df · 2020-08-05T14:44:10.000+03:00
Signed-off-by: Dmitry Sidorov &lt;dmitry.sidorov@intel.com&gt;
diff --git a/llvm-spirv/lib/SPIRV/SPIRVWriter.cpp b/llvm-spirv/lib/SPIRV/SPIRVWriter.cpp
@@ -563,10 +563,11 @@ SPIRVFunction *LLVMToSPIRV::transFunctionDecl(Function *F) {
       // Order of integer numbers in MD node follows the order of function
       // parameters on which we shall attach the appropriate decoration. Add
       // decoration only if MD value is not negative.
-      BM->addCapability(CapabilityFPGABufferLocationINTEL);
       int LocID = getMDOperandAsInt(BufferLocation, ArgNo);
-      if (LocID >= 0)
+      if (LocID >= 0) {
+        BM->addCapability(CapabilityFPGABufferLocationINTEL);
         BA->addDecorate(DecorationBufferLocationINTEL, LocID);
+      }
     }
   }
   if (Attrs.hasAttribute(AttributeList::ReturnIndex, Attribute::ZExt))

Original file line number	Diff line number	Diff line change
`@@ -563,10 +563,11 @@ SPIRVFunction LLVMToSPIRV::transFunctionDecl(Function F) {`
`563`	`563`	`// Order of integer numbers in MD node follows the order of function`
`564`	`564`	`// parameters on which we shall attach the appropriate decoration. Add`
`565`	`565`	`// decoration only if MD value is not negative.`
`566`		`- BM->addCapability(CapabilityFPGABufferLocationINTEL);`
`567`	`566`	`int LocID = getMDOperandAsInt(BufferLocation, ArgNo);`
`568`		`- if (LocID >= 0)`
	`567`	`+ if (LocID >= 0) {`
	`568`	`+ BM->addCapability(CapabilityFPGABufferLocationINTEL);`
`569`	`569`	`BA->addDecorate(DecorationBufferLocationINTEL, LocID);`
	`570`	`+ }`
`570`	`571`	`}`
`571`	`572`	`}`
`572`	`573`	`if (Attrs.hasAttribute(AttributeList::ReturnIndex, Attribute::ZExt))`