[PATCH] D78707: [MLIR] Add RecursiveSideEffects to Loops::ParallelOp.

[Theodore Popp via Phabricator via llvm-commits]

tpopp created this revision.
tpopp added a reviewer: herhut.
Herald added subscribers: llvm-commits, Kayjukh, frgossen, grosul1, Joonsoo, liufengdb, lucyrfox, mgester, arpith-jacob, nicolasvasilache, antiagainst, shauheen, jpienaar, rriddle, mehdi_amini.
Herald added a reviewer: nicolasvasilache.
Herald added a project: LLVM.

This is to specify that ParallelOp does not have side effects on its own
but has the effects of all operations executed in its region.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78707

Files:
  mlir/include/mlir/Dialect/LoopOps/LoopOps.td


Index: mlir/include/mlir/Dialect/LoopOps/LoopOps.td
===================================================================
--- mlir/include/mlir/Dialect/LoopOps/LoopOps.td
+++ mlir/include/mlir/Dialect/LoopOps/LoopOps.td
@@ -259,6 +259,7 @@
 def ParallelOp : Loop_Op<"parallel",
     [AttrSizedOperandSegments,
      DeclareOpInterfaceMethods<LoopLikeOpInterface>,
+     RecursiveSideEffects,
      SingleBlockImplicitTerminator<"YieldOp">]> {
   let summary = "parallel for operation";
   let description = [{


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78707.259530.patch
Type: text/x-patch
Size: 508 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200423/6d28a2ec/attachment.bin>