[PATCH] D88353: [BasicAA] Integrate MaxObjSize for NoAlias
Atmn Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 14 12:15:29 PDT 2020
atmnpatel added a comment.
I think that phrase ("w/o maxobjsize deduction") could be a bit misleading, IINM, when @jdoerfert says w/o maxobjsize deduction he means without having the functionality built into the attributor to update the maximum object size estimate. So in the patch D87975 <https://reviews.llvm.org/D87975>, there's a crude over-approximation for the maxobjsize (getPointerMaxObjSize) and the numbers are for just using that over-approximation without including the changes in D87978 <https://reviews.llvm.org/D87978> that refine it, so we would still need the maxobjsize attribute first in order to do this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88353/new/
https://reviews.llvm.org/D88353
More information about the llvm-commits
mailing list