This is an archive of the discontinued LLVM Phabricator instance.

[esan|cfrag] Handle binaries built with -esan-small-binary
ClosedPublic

Authored by zhaoqin on Jul 5 2016, 2:46 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

zhaoqin updated this revision to Diff 62795.Jul 5 2016, 2:46 PM
zhaoqin retitled this revision from to [esan|cfrag] Handle binaries built with -esan-small-binary.
zhaoqin updated this object.
zhaoqin added a reviewer: aizatsky.
zhaoqin added subscribers: vitalybuka, zhaoqin, kcc and 3 others.
aizatsky requested changes to this revision.Jul 6 2016, 11:35 AM
aizatsky edited edge metadata.
aizatsky added inline comments.
lib/esan/cache_frag.cpp
102 ↗(On Diff #62795)

Add comments regarding small binary to the struct.

Plus, ideally, add a simple HasFieldInfo() method to the struct.

This revision now requires changes to proceed.Jul 6 2016, 11:35 AM
zhaoqin updated this revision to Diff 62934.Jul 6 2016, 12:31 PM
zhaoqin edited edge metadata.

add comment and hasAuxFieldInfo method

zhaoqin marked an inline comment as done.Jul 6 2016, 12:33 PM
aizatsky accepted this revision.Jul 6 2016, 2:15 PM
aizatsky edited edge metadata.
aizatsky added inline comments.
lib/esan/cache_frag.cpp
36 ↗(On Diff #62934)

const

This revision is now accepted and ready to land.Jul 6 2016, 2:15 PM
zhaoqin updated this revision to Diff 63019.Jul 6 2016, 8:08 PM
zhaoqin edited edge metadata.

rename -esan-small-binary to -esan-aux-field-info

zhaoqin updated this object.Jul 6 2016, 8:11 PM
This revision was automatically updated to reflect the committed changes.