[Mlir-commits] [mlir] [mlir][amdgpu] Add amdgpu.make_dma_descriptor (PR #169407)

[Jakub Kuderski]

================
@@ -50,11 +50,51 @@ struct AMDGPUInlinerInterface final : DialectInlinerInterface {
 };
 } // namespace
 
+static ParseResult
+parseDynamicIndex(OpAsmParser &parser,
+                  std::optional<OpAsmParser::UnresolvedOperand> &dynamicSize,
+                  IntegerAttr &staticSize) {
+
+  int64_t staticVal;
+  OptionalParseResult parseResult = parser.parseOptionalInteger(staticVal);
+  if (parseResult.has_value()) {
----------------
kuhar wrote:

nit: why do we need this as a local variable, instead of using in the if condition like before?

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