[PATCH] D18784: [PGO] Remove redundant VP runtime calls
Mehdi AMINI via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 13 09:47:06 PDT 2016
joker.eph accepted this revision.
joker.eph added a reviewer: joker.eph.
joker.eph added a comment.
This revision is now accepted and ready to land.
LGTM (see one inline comment)
================
Comment at: test/Transforms/ADCE/profiling-calls.ll:8
@@ +7,3 @@
+define i32 @foo() {
+; CHECK-NOT: __llvm_profile_instrument_target
+entry:
----------------
It would be better with either:
- a comment like one line `verify that a call to to instrument a constant is deleted`.
- renaming the test file "delete-profiling-calls-to-constant.ll"
http://reviews.llvm.org/D18784
More information about the llvm-commits
mailing list