[PATCH] D149137: [Assignment Tracking] Check getTypeSizeInBits result for scalable vector types

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/lib/IR/DebugInfo.cpp:1948
+static std::optional<AssignmentInfo>
+getScalarAssignmentInfo(const DataLayout &DL, const Value *Dest, Type *StoredTy) {
+  TypeSize SizeInBits = DL.getTypeSizeInBits(StoredTy);
----------------
Is the use of "Scalar" here accurate? given it's only scalable vectors that are being rejected.

Only a suggestion but what about changing `getAssignmentInfoImpl` to take a `TypeSize` instead of an `uint64_t` and then reject scalable sizes within that? This way if/when scalable support is required the structure will already be in place.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149137/new/

https://reviews.llvm.org/D149137