[PATCH] D52243: [ConstHoist] Do not rebase single (or few) dependent constant

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 25 05:54:42 PDT 2018


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

LGTM, with one minor Nit.



================
Comment at: test/CodeGen/Thumb/consthoist-single-dependent.ll:60
+; dependent of base constant -2.
+; This test is also covered by https://reviews.llvm.org/D52257.
+
----------------
You can remove this phab reference, or point to the test added there.


Repository:
  rL LLVM

https://reviews.llvm.org/D52243





More information about the llvm-commits mailing list