[clang] [CIR] Upstream support for accessing structure members (PR #136383)

[Erich Keane via cfe-commits]

================
@@ -309,6 +309,37 @@ RecordType::computeStructAlignment(const mlir::DataLayout &dataLayout) const {
   return recordAlignment;
 }
 
+uint64_t RecordType::getElementOffset(const ::mlir::DataLayout &dataLayout,
+                                      unsigned idx) const {
+  assert(idx < getMembers().size() && "access not valid");
+
+  // All union elements are at offset zero.
+  if (isUnion() || idx == 0)
+    return 0;
+
+  assert(isComplete() && "Cannot get layout of incomplete records");
+  assert(idx < getNumElements());
+  llvm::ArrayRef<mlir::Type> members = getMembers();
+
+  unsigned offset = 0;
+
+  for (unsigned i = 0, e = idx; i != e; ++i) {
----------------
erichkeane wrote:

Is there good reason to not use range-for here? Something like:


`for (mlir::Type ty : std::make_range(members.begin(), std::next(members.begin(), idx)))` (though there is perhaps some ADT stuff to do a better job of this)

https://github.com/llvm/llvm-project/pull/136383