[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