[PATCH] D99592: [PoC][Clang] Use TypeSize instead of uint64_t for getTypeAllocSize().

2021-03-31 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai updated this revision to Diff 334590. HsiangKai added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99592/new/ https://reviews.llvm.org/D99592 Files: clang/lib/CodeGen/CGCall.cpp

[PATCH] D99592: [PoC][Clang] Use TypeSize instead of uint64_t for getTypeAllocSize().

2021-03-30 Thread Sander de Smalen via Phabricator via cfe-commits
sdesmalen added inline comments. Comment at: clang/lib/CodeGen/CGCall.cpp:2856 Address AddrToStoreInto = Address::invalid(); -if (SrcSize <= DstSize) { +if (llvm::TypeSize::isKnownLE(SrcSize, DstSize)) { AddrToStoreInto =

[PATCH] D99592: [PoC][Clang] Use TypeSize instead of uint64_t for getTypeAllocSize().

2021-03-30 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai created this revision. HsiangKai added reviewers: craig.topper, frasercrmck, rogfer01. Herald added a subscriber: StephenFan. HsiangKai requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The return type of getTypeAllocSize() is