[llvm-commits] [llvm] r101180 - /llvm/trunk/include/llvm/MC/MCDisassembler.h

[Gabor Greif]

On 13 Apr., 23:36, Sean Callanan <scalla... at apple.com> wrote:
> Author: spyffe
> Date: Tue Apr 13 16:36:35 2010
> New Revision: 101180
>
> URL:http://llvm.org/viewvc/llvm-project?rev=101180&view=rev
> Log:
> Quick fix for build errors caused by undefined
> NULL.
>
> Modified:
>     llvm/trunk/include/llvm/MC/MCDisassembler.h
>
> Modified: llvm/trunk/include/llvm/MC/MCDisassembler.h
> URL:http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCDisa...
> =========================================================================== ===
> --- llvm/trunk/include/llvm/MC/MCDisassembler.h (original)
> +++ llvm/trunk/include/llvm/MC/MCDisassembler.h Tue Apr 13 16:36:35 2010
> @@ -52,7 +52,7 @@
>    /// @return         - An array of instruction information, with one entry for
>    ///                   each MCInst opcode this disassembler returns.
>    ///                   NULL if there is no info for this target.
> -  virtual EDInstInfo   *getEDInfo() const { return NULL; }
> +  virtual EDInstInfo   *getEDInfo() const { return (EDInstInfo*)0; }

Hi Sean,

IIRC the cast is not needed here, "return 0;" will do perfectly well.

Cheers,

    Gabor

>  };
>
>  } // namespace llvm
>
> _______________________________________________
> llvm-commits mailing list
> llvm-comm... at cs.uiuc.eduhttp://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits