jansvoboda11 marked an inline comment as done. jansvoboda11 added inline comments.
================ Comment at: clang/include/clang/Basic/DirectoryEntry.h:211 + + static constexpr int NumLowBitsAvailable = 3; +}; ---------------- benlangmuir wrote: > I suggest not hard-coding it if you can get away with it; maybe > `PointerLikeTypeTraits<clang::DirectoryEntryRef::MapEntry > *>::NumLowBitsAvailable`? I think 3 could be wrong on a 32-bit platform for > example. Good idea, updated. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154905/new/ https://reviews.llvm.org/D154905 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
