[Mlir-commits] [mlir] [mlir][spirv] Expand support for TOSA Extended Instruction Set (00100… (PR #176908)

[Jakub Kuderski]

================
@@ -556,11 +557,17 @@ static void emitAttributeSerialization(const Attribute &attr,
     os << tabs << "    return failure();\n";
     os << tabs << "  }\n";
     os << tabs << formatv("  {0}.push_back(attrTypeID);\n", operandList);
-  } else if (attr.getAttrDefName() == "SPIRV_TensorArmAxisAttr") {
+  } else if (attr.getAttrDefName() == "SPIRV_TensorArmAxisAttr" ||
+             attr.getAttrDefName() == "SPIRV_BoolConstAttr") {
----------------
kuhar wrote:

Can you put these in an array and check for with `llvm::is_contained`? Also, let's keep the names in the lexicographic order.

https://github.com/llvm/llvm-project/pull/176908