public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug c++/87628] Redundant check of pointer when operator delete is called [not found] <bug-87628-4@http.gcc.gnu.org/bugzilla/> @ 2022-09-21 4:44 ` hiraditya at msn dot com 2022-09-21 5:00 ` hiraditya at msn dot com ` (2 subsequent siblings) 3 siblings, 0 replies; 4+ messages in thread From: hiraditya at msn dot com @ 2022-09-21 4:44 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87628 --- Comment #3 from AK <hiraditya at msn dot com> --- Still happening with gcc trunk. https://godbolt.org/z/5K94665GK ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug c++/87628] Redundant check of pointer when operator delete is called [not found] <bug-87628-4@http.gcc.gnu.org/bugzilla/> 2022-09-21 4:44 ` [Bug c++/87628] Redundant check of pointer when operator delete is called hiraditya at msn dot com @ 2022-09-21 5:00 ` hiraditya at msn dot com 2023-05-17 17:53 ` hiraditya at msn dot com 2023-05-17 18:14 ` hiraditya at msn dot com 3 siblings, 0 replies; 4+ messages in thread From: hiraditya at msn dot com @ 2022-09-21 5:00 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87628 --- Comment #4 from AK <hiraditya at msn dot com> --- Seems like clang now added the check: $ clang++ -Oz -fno-exceptions if_delete(char*): # @if_delete(char*) test rdi, rdi jne operator delete(void*)@PLT # TAILCALL ret ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug c++/87628] Redundant check of pointer when operator delete is called [not found] <bug-87628-4@http.gcc.gnu.org/bugzilla/> 2022-09-21 4:44 ` [Bug c++/87628] Redundant check of pointer when operator delete is called hiraditya at msn dot com 2022-09-21 5:00 ` hiraditya at msn dot com @ 2023-05-17 17:53 ` hiraditya at msn dot com 2023-05-17 18:14 ` hiraditya at msn dot com 3 siblings, 0 replies; 4+ messages in thread From: hiraditya at msn dot com @ 2023-05-17 17:53 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87628 --- Comment #5 from AK <hiraditya at msn dot com> --- As per: https://en.cppreference.com/w/cpp/memory/new/operator_delete """ In all cases, if ptr is a null pointer, the standard library deallocation functions do nothing. If the pointer passed to the standard library deallocation function was not obtained from the corresponding standard library allocation function, the behavior is undefined. """ So it should be fine to remove the check `if(p)` ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug c++/87628] Redundant check of pointer when operator delete is called [not found] <bug-87628-4@http.gcc.gnu.org/bugzilla/> ` (2 preceding siblings ...) 2023-05-17 17:53 ` hiraditya at msn dot com @ 2023-05-17 18:14 ` hiraditya at msn dot com 3 siblings, 0 replies; 4+ messages in thread From: hiraditya at msn dot com @ 2023-05-17 18:14 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87628 --- Comment #6 from AK <hiraditya at msn dot com> --- Opened a bug for clang as well: https://github.com/llvm/llvm-project/issues/62783 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-17 18:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-87628-4@http.gcc.gnu.org/bugzilla/>
2022-09-21 4:44 ` [Bug c++/87628] Redundant check of pointer when operator delete is called hiraditya at msn dot com
2022-09-21 5:00 ` hiraditya at msn dot com
2023-05-17 17:53 ` hiraditya at msn dot com
2023-05-17 18:14 ` hiraditya at msn dot com
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).