[patch] Delete dead code in lib/ProfileData

Duncan P. N. Exon Smith dexonsmith at apple.com
Sun Aug 31 15:56:16 PDT 2014


> On 2014 Aug 14, at 22:16, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> 
> A code review recently found a bug that in my patch in a change it
> made to lib/ProfileData. I was surprised that no test found it. On a
> closer look I realized the code was dead.
> 
> Normally I would just delete it, but since this is a fairly new
> library this might have been just an oversight.
> 
> Is this something that should be in use by tools/llvm-profdata? If it
> instead it is something that is intended for future use, it is
> probably a good idea to delete it for now and bring it back when it is
> actually used.

I think Justin is coordinating this now that Alex's internship has
completed.



More information about the llvm-commits mailing list