bricknerb wrote:

Thanks for doing this!
FWIW, I think splitting this to smaller PRs would be easier. For example:
1. `getFieldIndex()` inlining.
2. Passing fields instead of their indexes and iterate over fields.
3. Use `std::next()` instead of defining a `Next` variable.
4. Rename `isNoUniqueAddress` to `isNonVirtualBaseType`.

https://github.com/llvm/llvm-project/pull/122029
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to