[PATCH] D62888: [NewPM] Port Sancov
    Chandler Carruth via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Jun 10 17:37:36 PDT 2019
    
    
  
chandlerc requested changes to this revision.
chandlerc added a comment.
This revision now requires changes to proceed.
I would just change this to have the module pass loop over the functions -- that seems like it'll be much cleaner.
As it is, I'm not seeing where the loop actually happens. But rather than trying to figure that out, just seems better to turn it into an explicit loop. That way you can get rid of all the check analysis overhead I think.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62888/new/
https://reviews.llvm.org/D62888
    
    
More information about the cfe-commits
mailing list