Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20697 )
Change subject: IMPALA-11805: Fix LLVM memory manager bytes allocated ...................................................................... IMPALA-11805: Fix LLVM memory manager bytes allocated Previously the bytes_allocated function would undercount memory allocated by the LLVM memory manager, as it allocates memory in full pages but bytes_allocated only tracked the actual storage requested. Implements bytes_allocated in the SectionMemoryManager to access the actual storage size allocated. Change-Id: I5a0193a1694db0718d75651dc2b2f9f92c0d6064 Reviewed-on: http://gerrit.cloudera.org:8080/20697 Reviewed-by: Yida Wu <[email protected]> Reviewed-by: Csaba Ringhofer <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/codegen/llvm-codegen-cache-test.cc M be/src/codegen/llvm-codegen-test.cc M be/src/codegen/mcjit-mem-mgr.h M be/src/thirdparty/llvm/SectionMemoryManager.cpp M be/src/thirdparty/llvm/SectionMemoryManager.h 5 files changed, 40 insertions(+), 22 deletions(-) Approvals: Yida Wu: Looks good to me, but someone else must approve Csaba Ringhofer: Looks good to me, approved Michael Smith: Verified -- To view, visit http://gerrit.cloudera.org:8080/20697 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5a0193a1694db0718d75651dc2b2f9f92c0d6064 Gerrit-Change-Number: 20697 Gerrit-PatchSet: 14 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]>
