[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