[PATCH] D50339: Consistently use MemoryLocation::UnknownSize to indicate unknown access size
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 20 10:23:02 PDT 2018
MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D50339#1206109, @kparzysz wrote:
> The practical application of this is to indicate aliasing with a given memory object, but without specifying exactly which part of the object is accessed (since it may be unknown). Dropping memory operands altogether will alias the operation with all other objects, so it will be correct, but overly conservative.
Okay, it sounded to me like you are fixing a correctness problem, but if this is about allowing more information in the mem operands I'm fine with it.
Repository:
rL LLVM
https://reviews.llvm.org/D50339
More information about the llvm-commits
mailing list