[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