[PATCH] D112128: [NARY-REASSOCIATE][NFC] Simplify min/max handling

Evgeniy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 20 03:04:05 PDT 2021


ebrevnov created this revision.
Herald added a subscriber: hiraditya.
ebrevnov requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

In order to explore different variants of reassociation current implementation uses "swap in a loop" approach. Unfortunately, the implementation is more complicated than it could be. This is an attempt to streamline the code. New approach is to extract core functionality into a helper function and call it explicitly as many times as required.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D112128

Files:
  llvm/lib/Transforms/Scalar/NaryReassociate.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112128.380886.patch
Type: text/x-patch
Size: 5169 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211020/dd1cd119/attachment.bin>


More information about the llvm-commits mailing list