aeubanks added inline comments.
================ Comment at: clang/lib/CodeGen/Address.h:28 -// We try to save some space by using 6 bits over two PointerIntPairs to store -// the alignment. However, some arches don't support 3 bits in a PointerIntPair -// so we fallback to storing the alignment separately. -template <typename T, bool = alignof(llvm::Value *) >= 8> class AddressImpl {}; - -template <typename T> class AddressImpl<T, false> { +class AddressImpl { llvm::PointerIntPair<llvm::Value *, 1, bool> PointerAndKnownNonNull; ---------------- we should remove this class altogether Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144686/new/ https://reviews.llvm.org/D144686 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits