[libc-commits] [PATCH] D141041: [libc] Mark public C++ functions noexcept

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Jan 6 01:10:42 PST 2023


gchatelet abandoned this revision.
gchatelet added a subscriber: michaelrj.
gchatelet added a comment.

In D141041#4028944 <https://reviews.llvm.org/D141041#4028944>, @abrachet wrote:

> These aren't the public headers, so this is only marking it noexcept in a context where we likely already build without exceptions. We also don't every query if a function is noexcept in the code base either for the same reason. So I'm not sure what this is accomplishing.

Correct. @michaelrj patch <https://reviews.llvm.org/D141095> is the correct approach in this case then.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141041/new/

https://reviews.llvm.org/D141041



More information about the libc-commits mailing list