================
@@ -72,6 +74,29 @@ getOmpObjectSymbol(const Fortran::parser::OmpObject 
&ompObject) {
   return sym;
 }
 
+static Fortran::semantics::Symbol *
+getOmpObjectSymbol(const Fortran::parser::OmpObject &ompObject) {
----------------
ergawy wrote:

Instead of having this and the above functions, can we have one function with a 
`bool getParentObjWhenApplicable` argument?

I am suggesting this because almost all the logic is repeated with the 
exception of the `StructureComponent` case, right?

https://github.com/llvm/llvm-project/pull/81511
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to