[PATCH] D11780: -Wdeprecated: Use noexcept rather than throw() where supported

David Blaikie dblaikie at gmail.com
Wed Aug 5 17:24:02 PDT 2015


On Wed, Aug 5, 2015 at 2:36 PM, Alexey Samsonov <vonosmas at gmail.com> wrote:

> samsonov added a comment.
>
> The correct header for this is
> `lib/sanitizer_common/sanitizer_internal_defs.h`.


Done


> BTW we already have `NOTHROW` define there, but it does smth. different.
>

I updated the side of NOTHROW that defines it as "throw()" to use the
NOEXCEPT macro (so it'll use the non-deprecated noexcept, where available).


>
>
> ================
> Comment at: lib/tsan/rtl/tsan_new_delete.cc:39
> @@ +38,3 @@
> +#else
> +#define LLVM_NOEXCEPT LLVM_NOEXCEPT
> +#endif
> ----------------
> Huh?
>

Ah, probably did that accidentally when I s/throw()/NOEXCEPT/ in the file
to try to update all the existing sites.


>
>
> http://reviews.llvm.org/D11780
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150805/e41dbb75/attachment.html>


More information about the llvm-commits mailing list