[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