LGTM!

~Aaron

On Mon, May 25, 2015 at 4:26 PM, Richard <[email protected]> wrote:
> diff -U9999
>
>
> http://reviews.llvm.org/D10008
>
> Files:
>   lib/Analysis/BodyFarm.cpp
>
> Index: lib/Analysis/BodyFarm.cpp
> ===================================================================
> --- lib/Analysis/BodyFarm.cpp
> +++ lib/Analysis/BodyFarm.cpp
> @@ -36,10 +36,7 @@
>    // returns void.
>    const FunctionProtoType *FT =
>    BPT->getPointeeType()->getAs<FunctionProtoType>();
> -  if (!FT || !FT->getReturnType()->isVoidType() || FT->getNumParams() != 0)
> -    return false;
> -
> -  return true;
> +  return FT && FT->getReturnType()->isVoidType() && FT->getNumParams() == 0;
>  }
>
>  namespace {
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to