llvm dev - Feb 2011 - [LLVMdev] PR9112

Hello,

This simple patch fixes PR9112:

Index: lib/Analysis/ValueTracking.cpp
==================================================================---
lib/Analysis/ValueTracking.cpp      (revision 125281)
+++ lib/Analysis/ValueTracking.cpp      (working copy)
@@ -593,6 +593,8 @@
    // Otherwise take the unions of the known bit sets of the operands,
    // taking conservative care to avoid excessive recursion.
    if (Depth < MaxDepth - 1 && !KnownZero && !KnownOne) {
+      if (!P->getNumIncomingValues())
+        return;
      KnownZero = APInt::getAllOnesValue(BitWidth);
      KnownOne = APInt::getAllOnesValue(BitWidth);
      for (unsigned i = 0, e = P->getNumIncomingValues(); i != e; ++i) {

-- 
Jakub Staszak

Committed in r125319, thank you! For future reference, patches are supposed
to be mailed to llvm-commits instead.

Nick

On 10 February 2011 15:11, Jakub Staszak <kubastaszak at gmail.com> wrote:
> Hello,
>
> This simple patch fixes PR9112:
>
> Index: lib/Analysis/ValueTracking.cpp
> ==================================================================> ---
lib/Analysis/ValueTracking.cpp      (revision 125281)
> +++ lib/Analysis/ValueTracking.cpp      (working copy)
> @@ -593,6 +593,8 @@
>    // Otherwise take the unions of the known bit sets of the operands,
>    // taking conservative care to avoid excessive recursion.
>    if (Depth < MaxDepth - 1 && !KnownZero && !KnownOne)
{
> +      if (!P->getNumIncomingValues())
> +        return;
>      KnownZero = APInt::getAllOnesValue(BitWidth);
>      KnownOne = APInt::getAllOnesValue(BitWidth);
>      for (unsigned i = 0, e = P->getNumIncomingValues(); i != e; ++i) {
>
> --
> Jakub Staszak
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20110210/3989058e/attachment.html>