[PATCH] D18784: [PGO] Remove redundant VP runtime calls

[Betul Buyukkurt via llvm-commits]

These are not dead per say but rather redundant. I do not think any instrumentation around target values which would get resolved into a constant should stay in the final instrumented binary. 

-Betul

-----Original Message-----
From: Mehdi AMINI [mailto:mehdi.amini at apple.com] 
Sent: Monday, April 04, 2016 10:14 PM
To: betulb at codeaurora.org; davidxl at google.com; mail at justinbogner.com
Cc: mehdi.amini at apple.com; llvm-commits at lists.llvm.org
Subject: Re: [PATCH] D18784: [PGO] Remove redundant VP runtime calls

joker.eph added a comment.

If these are dead calls, why don't we constant fold them away to a no-op? 
Also do we want a separate pass or could/should it be part of ADCE?


Repository:
  rL LLVM

http://reviews.llvm.org/D18784