[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