[PATCH] D49103: Lower llvm.objectsize earlier in our optimization pipeline

[George Burgess IV via Phabricator via llvm-commits]

george.burgess.iv added a comment.

> You mentioned a 1.9% degradation with the current placement. Did you happen to try any other locations in the pipeline?

I did not, though I'd be happy to do so if anyone has recommendations :)

Looking at this, it appears that we may be running this pass too early with LTO enabled. I'll update this patch shortly to rebase it + delay running this pass until link-time if we're going to be running some flavor of LTO, and will collect new accuracy numbers for ${large_project} soonish.

Any other numbers/vetting would be highly appreciated.


https://reviews.llvm.org/D49103