<div dir="ltr"><div>In the LoopStrengthReduce.cpp code mentioned here, the code behaves as it was intended to; it's the indentation that's wrong.<br><br></div>Dan<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 2, 2016 at 10:58 AM, Dan Liew <span dir="ltr"><<a href="mailto:dan@su-root.co.uk" target="_blank">dan@su-root.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Using the correct e-mail address this time.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> ```<br>
> /home/dsl11/dev/llvm-upstream/src/lib/Transforms/Scalar/LoopStrengthReduce.cpp:<br>
> In member function ‘void {anonymous}::Cost::RateRegister(const<br>
> llvm::SCEV*, llvm::SmallPtrSetImpl<const llvm::SCEV*>&, const<br>
> llvm::Loop*, llvm::ScalarEvolution&, llvm::DominatorTree&)’:<br>
><br>
> /home/dsl11/dev/llvm-upstream/src/lib/Transforms/Scalar/LoopStrengthReduce.cpp:943:3:<br>
> warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]<br>
><br>
>    if (!isa<SCEVUnknown>(Reg) &&<br>
><br>
>    ^~<br>
><br>
> /home/dsl11/dev/llvm-upstream/src/lib/Transforms/Scalar/LoopStrengthReduce.cpp:950:5:<br>
> note: ...this statement, but the latter is misleadingly<br>
><br>
> indented as if it is guarded by the ‘if’<br>
><br>
>      NumIVMuls += isa<SCEVMulExpr>(Reg) &&<br>
><br>
>      ^~~~~~~~~<br>
> ```<br>
</div></div></blockquote></div><br></div>