cpf icon indicating copy to clipboard operation
cpf copied to clipboard
verified publisher icon PrincetonUniversity

Make profilings optional

Open vgene opened this issue 4 years ago • 2 comments

Make LAMP and SpecPriv optional

  • [x] LAMP
  • [ ] SpecPriv

vgene avatar Feb 23 '21 17:02 vgene

Added option -lamp-ignore to ignore LAMP profile or to indicate that it doesn't exist in f7d44abb45a7a4e089c88d363b8c03e453d9a6da Querying LAMPLoadProfile for number of deps with this option enabled conservatively returns UINT_MAX.

TODOs:

  • Merge LAMP and LAMPLoad into one library (contains identical versions of same pass in each)
  • SpecPriv optional

QUESTIONS:

  • Why is LAMP run twice in our pipeline?
  • SmtxAA is the only pass that uses LAMP directly? Maybe it should be renamed and moved to SCAF at some point.

gchan510 avatar Feb 26 '21 07:02 gchan510

I use LAMP results in the aggressive inliner. And then it needs to run again for the post-inlining code.

sapostolakis avatar Feb 26 '21 10:02 sapostolakis