[PATCH] D110698: [fir] Update fir.call op

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
This revision is now accepted and ready to land.

LGTM. A nit and a question.



================
Comment at: flang/include/flang/Optimizer/Dialect/FIROps.td:2657
     [{
-      build($_builder, $_state,
-            SymbolRefAttr::get($_builder.getContext(), callee), results,
-            operands);
+      build($_builder, $_state, 
+          SymbolRefAttr::get($_builder.getContext(), callee), results,
----------------
Is the plan to move all non-trivial builders to cpp files but retain the trivial ones in td?


================
Comment at: flang/include/flang/Optimizer/Dialect/FIROps.td:2669
     operand_range getArgOperands() {
-      if (calleeAttr())
+      if (auto calling =
+          (*this)->getAttrOfType<SymbolRefAttr>(getCalleeAttrName()))
----------------
Nit: wouldn't calling become an unused variable and lead to a warning?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110698/new/

https://reviews.llvm.org/D110698