arphaman accepted this revision.
arphaman added inline comments.
This revision is now accepted and ready to land.
================
Comment at:
clang/lib/Tooling/DependencyScanning/DependencyScanningFilesystem.cpp:249
if (!Entry->getPPSkippedRangeMapping().empty() && PPSkipMappings)
- (*PPSkipMappings)[Result->getBufferPtr()] =
+ (*PPSkipMappings)[Result->Buffer->getBufferStart()] =
&Entry->getPPSkippedRangeMapping();
----------------
`Buffer` is private, does this work here, or do you need a `friend`? Either
way, LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89136/new/
https://reviews.llvm.org/D89136
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
