jyknight added a comment.
> As a result, clang/llvm should correctly
> optimize custom allocators like this without needing specialized
> knowledge going forward.
Not exactly: we still need knowledge of this in //clang//
(`isReplaceableGlobalAllocationFunction` was taught that the variant with a
durin42 added a comment.
FYI: this is ready for review.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152401/new/
https://reviews.llvm.org/D152401
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
durin42 edited the summary of this revision.
durin42 updated this revision to Diff 529617.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152401/new/
https://reviews.llvm.org/D152401
Files:
clang/lib/CodeGen/CGExprCXX.cpp
clang/test/CodeGenCXX/cx
durin42 created this revision.
Herald added subscribers: JDevlieghere, hiraditya.
Herald added a project: All.
durin42 requested review of this revision.
Herald added projects: clang, LLVM.
Herald added subscribers: llvm-commits, cfe-commits.
This allows us to remove most of the special knowledge