Page MenuHomePhabricator

Remove sample profile dependency to instcombine, which is not a analysis pass.
ClosedPublic

Authored by danielcdh on May 20 2016, 5:23 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

danielcdh updated this revision to Diff 58020.May 20 2016, 5:23 PM
danielcdh retitled this revision from to Remove sample profile dependency to instcombine, which is not a analysis pass..
danielcdh updated this object.
danielcdh added a reviewer: davidxl.
danielcdh added a subscriber: llvm-commits.
dnovillo edited edge metadata.May 24 2016, 6:32 AM

Now, if the sample profiler depends on instcombine, we'd have to schedule it after instcombine, right?

dnovillo accepted this revision.May 24 2016, 6:34 AM
dnovillo edited edge metadata.

Now, if the sample profiler depends on instcombine, we'd have to schedule it after instcombine, right?

If I read mail in order, I wouldn't be tempted to ask inane questions. You have indeed done this in another patch. Thanks!

LGTM

This revision is now accepted and ready to land.May 24 2016, 6:34 AM
This revision was automatically updated to reflect the committed changes.