================
@@ -4022,8 +4169,36 @@ LValue CodeGenFunction::EmitArraySubscriptExpr(const
ArraySubscriptExpr *E,
ArrayLV = EmitArraySubscriptExpr(ASE, /*Accessed*/ true);
else
ArrayLV = EmitLValue(Array);
+
auto *Idx = EmitIdxAfterBase(/*Promote*/true);
+ if (SanOpts.has(SanitizerKind::ArrayBounds)) {
----------------
bwendling wrote:
Thanks, @rapidsna. I agree with what you said here. (I also agree with
@rjmccall.) I don't want to make anymore changes to this patch though. As
@nathanchance pointed out, this is holding up Linux testing / development and I
want to submit it soon. I believe it's ready for submission.
To be honest, this patch was only meant as a bug fix, but snowballed out of all
proportions. However, I think it's in a state where we could submit this and
continue development from there.
I need to point out that this *isn't* the last patch in this feature. I'll have
more to follow. Can someone please just give approval so that we can get on
with life?
https://github.com/llvm/llvm-project/pull/73730
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
