[Mlir-commits] [flang] [mlir] [flang][mlir][OpenMP] Add linear modifier (val, ref, uval) (PR #187142)

[Tom Eccles]

================
@@ -1488,16 +1488,37 @@ bool ClauseProcessor::processIsDevicePtr(
   return clauseFound;
 }
 
-bool ClauseProcessor::processLinear(mlir::omp::LinearClauseOps &result) const {
+bool ClauseProcessor::processLinear(mlir::omp::LinearClauseOps &result,
+                                    bool isDeclareSimd) const {
   lower::StatementContext stmtCtx;
   return findRepeatableClause<
       omp::clause::Linear>([&](const omp::clause::Linear &clause,
                                const parser::CharBlock &) {
     auto &objects = std::get<omp::ObjectList>(clause.t);
     static std::vector<mlir::Attribute> typeAttrs;
+    static std::vector<mlir::Attribute> linearModAttrs;
 
-    if (!result.linearVars.size())
+    if (!result.linearVars.size()) {
----------------
tblah wrote:

When is this the case in correct operation? I wonder if it should be an assert but I'm not 100% sure.

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