[llvm] r232508 - Fix GCC -Wparentheses warning (& reformat now that the precedence is fixed)

Sanjoy Das sanjoy at playingwithpointers.com
Tue Mar 17 10:58:31 PDT 2015


Thanks!

On Tue, Mar 17, 2015 at 10:48 AM, David Blaikie <dblaikie at gmail.com> wrote:
> Author: dblaikie
> Date: Tue Mar 17 12:48:24 2015
> New Revision: 232508
>
> URL: http://llvm.org/viewvc/llvm-project?rev=232508&view=rev
> Log:
> Fix GCC -Wparentheses warning (& reformat now that the precedence is fixed)
>
> Benign warning (clang deliberately suppresses this case) but does
> regularly produce bad formatting, so it's nice to fix/reformat.
>
> Modified:
>     llvm/trunk/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp
>
> Modified: llvm/trunk/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp?rev=232508&r1=232507&r2=232508&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp (original)
> +++ llvm/trunk/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp Tue Mar 17 12:48:24 2015
> @@ -397,8 +397,8 @@ InductiveRangeCheck::create(InductiveRan
>      return nullptr;
>
>    assert(IndexSCEV && "contract with SplitRangeCheckCondition!");
> -  assert(!(RCKind & InductiveRangeCheck::RANGE_CHECK_UPPER) ||
> -         Length && "contract with SplitRangeCheckCondition!");
> +  assert((!(RCKind & InductiveRangeCheck::RANGE_CHECK_UPPER) || Length) &&
> +         "contract with SplitRangeCheckCondition!");
>
>    const SCEVAddRecExpr *IndexAddRec = dyn_cast<SCEVAddRecExpr>(IndexSCEV);
>    bool IsAffineIndex =
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list