[llvm] [FunctionAttrs] Add the "initializes" attribute inference (PR #97373)
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 10 15:50:46 PDT 2024
================
@@ -580,6 +582,208 @@ struct ArgumentUsesTracker : public CaptureTracker {
const SCCNodeSet &SCCNodes;
};
+// A struct of argument use: a Use and the offset it accesses. This struct
+// is to track uses inside function via GEP. If GEP has a non-constant index,
+// the Offset field is nullopt.
+struct ArgumentUse {
+ Use *U;
+ std::optional<int64_t> Offset;
+};
+
+// A struct of argument access info. "Unknown" accesses are the cases like
+// unrecognized instructions, instructions that have more than one use of
+// the argument, or volatile memory accesses. "Unknown" implies "IsClobber"
+// and an empty access range.
+// Write or Read accesses can be clobbers as well for example, a Load with
+// scalable type.
+struct ArgumentAccessInfo {
+ enum class AccessType : uint8_t { Write, Read, Unknown };
+ AccessType ArgAccessType;
+ bool IsClobber = false;
+ ConstantRangeList AccessRanges;
+};
+
+struct UsesPerBlockInfo {
+ SmallDenseMap<Instruction *, ArgumentAccessInfo, 4> Insts;
+ bool HasWrites = false;
+ bool HasClobber = false;
+};
+
+ArgumentAccessInfo GetArgmentAccessInfo(const Instruction *I,
+ const ArgumentUse &ArgUse,
+ const DataLayout &DL) {
+ auto GetTypeAccessRange =
+ [&DL](Type *Ty,
+ std::optional<int64_t> Offset) -> std::optional<ConstantRange> {
+ auto TypeSize = DL.getTypeStoreSize(Ty);
+ if (!TypeSize.isScalable() && Offset) {
+ int64_t Size = TypeSize.getFixedValue();
+ return ConstantRange(APInt(64, *Offset, true),
+ APInt(64, *Offset + Size, true));
+ }
+ return std::nullopt;
+ };
+ auto GetConstantIntRange =
+ [](Value *Length,
+ std::optional<int64_t> Offset) -> std::optional<ConstantRange> {
+ auto *ConstantLength = dyn_cast<ConstantInt>(Length);
+ if (ConstantLength && Offset)
+ return ConstantRange(
+ APInt(64, *Offset, true),
+ APInt(64, *Offset + ConstantLength->getSExtValue(), true));
+ return std::nullopt;
+ };
+ if (auto *SI = dyn_cast<StoreInst>(I)) {
----------------
aeubanks wrote:
check volatile? and test coverage seems to be missing
https://github.com/llvm/llvm-project/pull/97373
More information about the llvm-commits
mailing list