amaiorano wrote:

> Please also make the unused "operator delete" in CGFunctionInfo.h deleted, 
> since it shouldn't have any callers.
> 
> I think this never actually led to a real issue because of that operator 
> delete... it calls the right deallocation function even if it's technically 
> undefined. So the only way to spot an issue would be a sanitizer that 
> specifically checks for this. But I'm happy to "fix" this anyway.

Just to clarify, this was caught in an ASAN build of DXC, which is a fork of 
Clang. This may be reproducible with Clang by compiling with 
`-fsized-deallocation`.

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

Reply via email to