[llvm-commits] [llvm] r165708 - in /llvm/trunk: include/llvm/ include/llvm/Analysis/ lib/Analysis/ test/Analysis/DependenceAnalysis/

[Sebastian Pop]

On Thu, Oct 11, 2012 at 3:18 AM, Sebastian Pop <spop at codeaurora.org> wrote:
> On Thu, Oct 11, 2012 at 2:59 AM, Bill Wendling <wendling at apple.com> wrote:
>> Hi Sebastian,
>>
>> I'm getting this warning. Could you fix it please?
>>
>> /Volumes/Sandbox/llvm/llvm-clean.src/lib/Analysis/DependenceAnalysis.cpp:1164:32: warning: implicit truncation from 'int' to bitfield changes value from -5 to 3
>>       [-Wconstant-conversion]
>>     Result.DV[Level].Direction &= ~Dependence::DVEntry::GT;
>>                                ^  ~~~~~~~~~~~~~~~~~~~~~~~~
>> /Volumes/Sandbox/llvm/llvm-clean.src/lib/Analysis/DependenceAnalysis.cpp:1226:34: warning: implicit truncation from 'int' to bitfield changes value from -5 to 3
>>       [-Wconstant-conversion]
>>       Result.DV[Level].Direction &= ~Dependence::DVEntry::GT;
>>                                  ^  ~~~~~~~~~~~~~~~~~~~~~~~~
>
> I haven't seen this warning in my test: I was building with GCC.
> Preston, is this change ok with you?
>

My change wouldn't fit, as there are arrays indexed by the direction
and they expect it to be a value less than 8.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation