r313907 - Suppress Wsign-conversion for enums with matching underlying type

[Friedman, Eli via cfe-commits]

On 9/21/2017 12:58 PM, Erich Keane via cfe-commits wrote:
> Author: erichkeane
> Date: Thu Sep 21 12:58:55 2017
> New Revision: 313907
>
> URL: http://llvm.org/viewvc/llvm-project?rev=313907&view=rev
> Log:
> Suppress Wsign-conversion for enums with matching underlying type
>
> As reported here: https://bugs.llvm.org/show_bug.cgi?id=34692
>
> A non-defined enum with a backing type was always defaulting to
> being treated as a signed type. IN the case where it IS defined,
> the signed-ness of the actual items is used.
>
> This patch uses the underlying type's signed-ness in the non-defined
> case to test signed-comparision.
>
> Differential Revision: https://reviews.llvm.org/D38145
>
> Modified:
>      cfe/trunk/lib/Sema/SemaChecking.cpp

Missing testcase?

-El

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project