[PATCH] D70049: [CodeMoverUtils] Added an API to check if an instruction can be safely moved before another instruction.

[Whitney Tsang via Phabricator via llvm-commits]

Whitney updated this revision to Diff 229388.
Whitney edited the summary of this revision.
Whitney added a comment.

With this change, no instructions are considered safe to move across instructions that may throw, or call instructions that may not return or may synchronize, which is straighter than @jdoerfert suggestion. We can generalize it when needed. @jdoerfert What do you think?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70049

Files:
  llvm/include/llvm/Analysis/PostDominators.h
  llvm/include/llvm/Transforms/Utils/CodeMoverUtils.h
  llvm/lib/Analysis/PostDominators.cpp
  llvm/lib/Transforms/Utils/CMakeLists.txt
  llvm/lib/Transforms/Utils/CodeMoverUtils.cpp
  llvm/unittests/Transforms/Utils/CMakeLists.txt
  llvm/unittests/Transforms/Utils/CodeMoverUtilsTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70049.229388.patch
Type: text/x-patch
Size: 18845 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191114/f90c6b84/attachment.bin>