probinson added a comment. I had tried to do this in r333311 but some bots complained, so I reverted it and then didn't follow through. Thanks for doing this! When I tried it, I took advantage of existing tracking of line directives at the file level, so there shouldn't be a need to add a Line flag to PresumedLoc? What I did was something like this, in computeChecksum():
const SrcMgr::SLocEntry &Entry = SM.getSLocEntry(FID, &Invalid); if (Invalid || !Entry.isFile() || Entry.getFile().hasLineDirectives()) return None; Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60283/new/ https://reviews.llvm.org/D60283 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits