[PATCH] D78701: [mlir] Give each OpResult its own use list

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 04:49:37 PDT 2020


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

This revision removes the multi use-list to ensure that each result gets its own. This decision was made by doing some extensive benchmarking of programs that actually use multiple results. This results in a size increase of 1-word per result >1, but the common case of 1-result remains unaffected. A side benefit is that 0-result operations now shrink by 1-word.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78701

Files:
  mlir/include/mlir/IR/Operation.h
  mlir/include/mlir/IR/OperationSupport.h
  mlir/include/mlir/IR/UseDefLists.h
  mlir/include/mlir/IR/Value.h
  mlir/lib/IR/Operation.cpp
  mlir/lib/IR/OperationSupport.cpp
  mlir/lib/IR/Value.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78701.259519.patch
Type: text/x-patch
Size: 31164 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200423/9524b412/attachment.bin>


More information about the llvm-commits mailing list