Page MenuHomePhabricator

[esan|cfrag] Minor cfrag report adjustment
ClosedPublic

Authored by zhaoqin on Jun 14 2016, 2:36 PM.

Details

Summary

Reports the struct field access info only if it has been used at least
once.

Adds type printing size limit.

Formats the cache_frag.cpp with clang-format.

Diff Detail

Repository
rL LLVM

Event Timeline

zhaoqin updated this revision to Diff 60758.Jun 14 2016, 2:36 PM
zhaoqin retitled this revision from to [esan|cfrag] Report the struct info only if it has been used.
zhaoqin updated this object.
zhaoqin added a reviewer: bruening.
zhaoqin added subscribers: aizatsky, vitalybuka, zhaoqin and 3 others.
zhaoqin retitled this revision from [esan|cfrag] Report the struct info only if it has been used to [esan|cfrag] Minor cfrag report adjustment.Jun 14 2016, 3:14 PM
zhaoqin updated this object.
zhaoqin updated this revision to Diff 60771.Jun 14 2016, 3:14 PM

add type printing size limit

bruening added inline comments.Jun 14 2016, 8:27 PM
lib/esan/cache_frag.cpp
102 ↗(On Diff #60771)

Please make this a named constant

zhaoqin updated this revision to Diff 60834.Jun 15 2016, 7:47 AM
zhaoqin marked an inline comment as done.

PTAL

bruening accepted this revision.Jun 15 2016, 8:38 AM
bruening edited edge metadata.
This revision is now accepted and ready to land.Jun 15 2016, 8:38 AM
This revision was automatically updated to reflect the committed changes.