[PATCH] D84776: [CodeMoverUtils] Add more tests to check code motion safety using data dependence information.

rithik sharma via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 07:58:50 PDT 2020


RithikSharma marked 3 inline comments as done.
RithikSharma added inline comments.


================
Comment at: llvm/unittests/Transforms/Utils/CodeMoverUtilsTest.cpp:1006
+  std::unique_ptr<Module> M = parseIR(
+      C, R"(define dso_local i32 @dependence(i32* noalias %A, i32* noalias %B) {
+entry:
----------------
Whitney wrote:
> Whitney wrote:
> > remove dso_local
> did you forget to update your patch? cause dso_local is still there.
Updated, thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84776



More information about the llvm-commits mailing list