[PATCH] D45003: [Support] Make line-number cache robust against access patterns.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 4 16:13:03 PDT 2018


probinson added a comment.

In https://reviews.llvm.org/D45003#1055099, @graydon wrote:

> In https://reviews.llvm.org/D45003#1053430, @probinson wrote:
>
> > There's commentary in lib/MC/MCParser/AsmParser.cpp about the ungraceful degradation in SourceMgr's cache.  Would this help or simplify what AsmParser is doing?
>
>
> I believe this should eliminate the behaviour those comments describe (and thus remove the need for the code), though I feel a bit under-qualified tinkering with the code there itself. Would you like me to try, or shall I leave it to the owners of that file?


If you could have a go at it, that would be great.  It's fine to do it as a follow-up NFC patch.  Or file a bug and cc me.  It's not really my area either but I keep running across that comment and it has always bothered me, maybe enough to do something now.


Repository:
  rL LLVM

https://reviews.llvm.org/D45003





More information about the llvm-commits mailing list