[PATCH] D90484: FileManager: Add FileEntryRef::getDir, returning DirectoryEntryRef

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 30 14:52:21 PST 2020


dexonsmith added a comment.

Thanks again for the review, landed in 1b042de5b29af4869a77ecbc632029fba0313dec <https://reviews.llvm.org/rG1b042de5b29af4869a77ecbc632029fba0313dec>.



================
Comment at: clang/unittests/Basic/FileEntryTest.cpp:95
 
   OptionalFileEntryRefDegradesToFileEntryPtr M0;
   OptionalFileEntryRefDegradesToFileEntryPtr M1 = R1;
----------------
dexonsmith wrote:
> jansvoboda11 wrote:
> > Dead variable, do we need it?
> Thanks, nice catch, that looks vestigial. I'll delete that line in a follow-up.
Deleted this stray line in 94f537c6b2bb55c8e058a9989d02ab0d68a0c61a; turns out it was originally a copy/paste bug from ac49500cd0484e1b2dcf37fa4c0dade6f113c2c9.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90484/new/

https://reviews.llvm.org/D90484



More information about the cfe-commits mailing list