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

[Preston Briggs]

I think the attached patch will do the trick.

Thanks,
Preston


On Thu, Oct 11, 2012 at 1:42 AM, Sebastian Pop <spop at codeaurora.org> wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121011/e605a47f/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: da.patch
Type: application/octet-stream
Size: 1944 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121011/e605a47f/attachment.obj>