[PATCH] D80643: [CodeMoverUtils] Move OrderedInstructions to CodeMoverUtils

rithik sharma via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 01:36:38 PDT 2020


RithikSharma updated this revision to Diff 269415.
RithikSharma added a comment.

This update removes the OrderedInstructions class from CodeMoverUtils and also removes it's exposure to other transformation passes, both `localDominates`  and `domTreeLevelBefore` is made as static functions and are present in CodeMoverUtils.cpp. `OrderedInstructions::dominates()` has  been removed from CodeMoverUtils.cpp due to it's existance in `DominatorTree::dominates()`, the dominance test moved from `OrderedInstructionsTest.cpp` to `CodeMoverUtilsTest.cpp` is also removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80643

Files:
  llvm/include/llvm/Analysis/OrderedInstructions.h
  llvm/lib/Analysis/CMakeLists.txt
  llvm/lib/Analysis/OrderedInstructions.cpp
  llvm/lib/Transforms/Utils/CodeMoverUtils.cpp
  llvm/unittests/Analysis/CMakeLists.txt
  llvm/unittests/Analysis/OrderedInstructionsTest.cpp
  llvm/utils/gn/secondary/llvm/lib/Analysis/BUILD.gn
  llvm/utils/gn/secondary/llvm/unittests/Analysis/BUILD.gn

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80643.269415.patch
Type: text/x-patch
Size: 11375 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200609/ad5f7c80/attachment-0001.bin>


More information about the llvm-commits mailing list