================
@@ -1901,9 +1903,8 @@ class SourceManager : public 
RefCountedBase<SourceManager> {
 
   FileID getFileID(SourceLocation::UIntTy SLocOffset) const {
     // If our one-entry cache covers this offset, just return it.
-    if (isOffsetInFileID(LastFileIDLookup, SLocOffset))
+    if (SLocOffset >= LastLookupStartOffset && SLocOffset < 
LastLookupEndOffset)
----------------
erichkeane wrote:

Ow!  That is sad.  I guess 1927 says that `isOffsetInFileID` is a `very hot 
method`, so that tells me that this check succeeds comparatively rarely in THAT 
function.  Makes me wonder how valuable it is HERE, though I guess with all the 
work `slow` is doing, it matters...

https://github.com/llvm/llvm-project/pull/146782
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to