[llvm] [Analysis] Add Scalable field in MemoryLocation.h (PR #69716)
Harvin Iriawan via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 24 06:56:30 PDT 2023
================
@@ -176,14 +176,14 @@ static uint64_t getMinimalExtentFrom(const Value &V,
// If queried with a precise location size, we assume that location size to be
// accessed, thus valid.
if (LocSize.isPrecise())
- DerefBytes = std::max(DerefBytes, LocSize.getValue());
- return DerefBytes;
+ DerefBytes = std::max(DerefBytes, LocSize.getValue().getKnownMinValue());
+ return LocationSize(DerefBytes);
----------------
harviniriawan wrote:
so at the end, should I make it return `TypeSize::getFixed(DerefBytes)` instead of `TypeSize::get(DerefBytes, LocSize.isScalable())` ?
Because at first I was changing even `DerefBytes` to be of `TypeSize`
https://github.com/llvm/llvm-project/pull/69716
More information about the llvm-commits
mailing list