bnbarham accepted this revision. bnbarham added a comment. This revision is now accepted and ready to land.
Thanks 🙇 ================ Comment at: clang/unittests/Basic/FileManagerTest.cpp:381 EXPECT_TRUE(F1->isSameRef(*F1Again)); - EXPECT_TRUE(F1->isSameRef(*F1Redirect)); + EXPECT_FALSE(F1->isSameRef(*F1Redirect)); EXPECT_FALSE(F1->isSameRef(*F1Also)); ---------------- Bit weird that this test was checking for true 🤔. Seems like it isn't used outside of tests except for `isEqual` in `DenseMapInfo` though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131004/new/ https://reviews.llvm.org/D131004 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
