================
@@ -865,39 +865,55 @@ Address CIRGenFunction::getAddressOfBaseClass(
bool nullCheckValue, SourceLocation loc) {
assert(!path.empty() && "Base path should not be empty!");
+ CastExpr::path_const_iterator start = path.begin();
----------------
andykaylor wrote:
We're already passing path as a range into this function and in the call to
`computeNonVirtualBaseClassOffset`, which was a change from the classic codegen
and incubator implementations, and the LLVM iterator_range is quite limited.
The only thing I can see that we could do here is build a new range on lin 873
instead of doing it on line 880. We don't need `start` for anything else after
line 873, so we could get rid of it.
https://github.com/llvm/llvm-project/pull/159162
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
