This is an archive of the discontinued LLVM Phabricator instance.

Sync InstrProfData.inc between llvm and compiler-rt
ClosedPublic

Authored by ellis on Mar 19 2021, 1:06 PM.

Details

Summary

Fix difference in InstrProfData.inc between llvm and compiler-rt.

My understanding is that
llvm/include/llvm/ProfileData/InstrProfData.inc and
compiler-rt/include/profile/InstrProfData.inc need to stay identical.

diff llvm/include/llvm/ProfileData/InstrProfData.inc compiler-rt/include/profile/InstrProfData.inc

Diff Detail

Event Timeline

ellis created this revision.Mar 19 2021, 1:06 PM
ellis requested review of this revision.Mar 19 2021, 1:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 19 2021, 1:06 PM
ellis edited the summary of this revision. (Show Details)Mar 19 2021, 1:27 PM
lanza added subscribers: MaskRay, lanza.EditedMar 19 2021, 3:25 PM

The comment at the top says this is the master but this def seems right. @MaskRay did the compiler-rt change so I'll just tag him in.

davidxl accepted this revision.Mar 19 2021, 3:25 PM

lgtm

This revision is now accepted and ready to land.Mar 19 2021, 3:25 PM
ellis added a comment.Mar 19 2021, 3:32 PM

Thanks for accepting @davidxl
Would you mind landing this? I don't have commit access.

MaskRay accepted this revision.Mar 19 2021, 4:16 PM
This revision was landed with ongoing or failed builds.Mar 19 2021, 4:24 PM
This revision was automatically updated to reflect the committed changes.