[PATCH] D123197: Remove a few effectively-unused FileEntry APIs. NFC

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet added inline comments.


================
Comment at: clang/lib/Basic/FileManager.cpp:462
   UFE->Dir     = &DirInfo->getDirEntry();
-  UFE->UID     = NextFileUID++;
-  UFE->IsValid = true;
+  UFE->UID = NextFileUID++;
   UFE->File.reset();
----------------
sammccall wrote:
> kadircet wrote:
> > nit: revert formatting
> `arc` insists on this change here, because it's in the blast radius of the next line and clang-format isn't configured to allow this formatting.
> 
> Unless you feel strongly I'd rather not fight the linter when it's not really wrong.
> (Happy to reformat the other lines to match though)
> Unless you feel strongly I'd rather not fight the linter when it's not really wrong.
> (Happy to reformat the other lines to match though)

I was suggesting this for preserving history purposes, in theory all of this is old code with intricate semantics so it would be nice to not add more commits to the history chain here. I think it should be possible to just `arc diff --nolint` (or push without updating the diff in phabricator, which you already probably know, but just in case).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123197