[libcxx] r185265 - Add operators to make launch a bitmask type. Searched all of the standard, and libc++ to see if this error occurred elsewhere and didn't see any other place. This fixes http://llvm.org/bugs/show_bug.cgi?id=16207

Marshall Clow mclow.lists at gmail.com
Mon Jul 1 06:15:46 PDT 2013


On Jun 29, 2013, at 11:38 AM, Howard Hinnant <hhinnant at apple.com> wrote:

> +inline _LIBCPP_INLINE_VISIBILITY
> +_LIBCPP_CONSTEXPR
> +launch
> +operator~(launch __x)
> +{
> +    return static_cast<launch>(~static_cast<__launch_underlying_type>(__x));
> +}

This may cause problems down the line, because it sets bits in launch that are not defined to have any meaning.
I ran into this kind of stuff when trying to use UBSan.

To prevent this problem, you can AND the result with 3, clearing all the unused bits.

> +    return static_cast<launch>(~static_cast<__launch_underlying_type>(__x) & 3);

(Yeah, I hate the naked '3' there too.)

-- Marshall

Marshall Clow     Idio Software   <mailto:mclow.lists at gmail.com>

A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait).
        -- Yu Suzuki





More information about the cfe-commits mailing list