[PATCH] D43511: [dsymutil] Be smarter in caching calls to realpath

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere added inline comments.


================
Comment at: llvm/tools/dsymutil/DwarfLinker.cpp:1949
+            std::string File;
+            bool FoundFile =
+              LT->getFileNameByIndex(FileNum, "",
----------------
aprantl wrote:
> Might be more efficient to wrap this in an
> ```
> #ifndef NDEBUG
> #endif
> ```
> block.
I gave it a shot but I'm not convinced it's better :-) 


https://reviews.llvm.org/D43511