[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