[llvm-commits] [llvm] r48084 - in /llvm/trunk: lib/Transforms/IPO/PruneEH.cpp test/Transforms/PruneEH/unwindto.ll

Nick Lewycky nicholas at mxc.ca
Sun Mar 9 10:13:11 PDT 2008


Duncan Sands wrote:
> Hi Nick,
> 
>> +      } else
>> +        couldUnwind = true;
> 
> a stylistic point: please use
> 	} else {
> 		couldUnwind = true;
> 	}
> when the previous if block is in {}.
> 
>> +        } else if (!CI->doesNotThrow())
>> +          couldUnwind = true;
> 
> Likewise.

Done. Thanks for reviewing it!

Nick



More information about the llvm-commits mailing list