[PATCH] D30130: ArgList: cache index ranges containing arguments with each ID
    Reid Kleckner via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Apr 10 11:06:41 PDT 2017
    
    
  
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
I agree with your analysis on the mailing list, let's go with this change as is. Keeping maps of range is super simple. Keeping a map of lists that need to be merged on demand makes temporary data structures, which might be slow, and isn't a clear win.
Repository:
  rL LLVM
https://reviews.llvm.org/D30130
    
    
More information about the llvm-commits
mailing list