[PATCH] D74174: [MLIR] Allow Loop dialect IfOp and ForOp to define values

[Nagy Mostafa via Phabricator via llvm-commits]

nmostafa updated this revision to Diff 243660.
nmostafa marked an inline comment as done.
nmostafa edited the summary of this revision.
nmostafa added a comment.

- Added custom printer for YieldOp
- Added tests to verify loop.parallel yield has no operands. Negative test for that as well.
- Updated td description for loop.parallel: yield without opreands
- Updates to commit message


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

https://reviews.llvm.org/D74174

Files:
  mlir/examples/toy/Ch6/mlir/LowerToLLVM.cpp
  mlir/examples/toy/Ch7/mlir/LowerToLLVM.cpp
  mlir/include/mlir/Dialect/LoopOps/LoopOps.td
  mlir/include/mlir/IR/OpImplementation.h
  mlir/lib/Conversion/AffineToStandard/AffineToStandard.cpp
  mlir/lib/Conversion/GPUToSPIRV/ConvertGPUToSPIRV.cpp
  mlir/lib/Dialect/LoopOps/LoopOps.cpp
  mlir/lib/Parser/Parser.cpp
  mlir/test/Dialect/Linalg/parallel_loops.mlir
  mlir/test/Dialect/Loops/invalid.mlir
  mlir/test/Dialect/Loops/ops.mlir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74174.243660.patch
Type: text/x-patch
Size: 33936 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200210/511f2cba/attachment.bin>