================
@@ -2228,9 +2228,6 @@ Address 
CIRGenItaniumCXXABI::initializeArrayCookie(CIRGenFunction &cgf,
                                                    QualType elementType) {
   assert(requiresArrayCookie(e));
 
-  // TODO: When sanitizer support is implemented, we'll need to
-  // get the address space from `newPtr`.
----------------
aobolensk wrote:

Let's leave it just in case it has some value for the history. I assumed that 
we have implemented address spaces anyway, so this comment is kind of redundant

Probably, maintainers remember something about that. Let's wait for their 
feedback

https://github.com/llvm/llvm-project/pull/231367
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to