[llvm-commits] [llvm] r71210 - /llvm/trunk/lib/Transforms/Scalar/InstructionCombining.cpp

Misha Brukman brukman at gmail.com
Fri May 8 09:13:29 PDT 2009


On Fri, May 8, 2009 at 2:47 AM, Nick Lewycky <nicholas at mxc.ca> wrote:

> Author: nicholas
> Date: Fri May  8 01:47:37 2009
> New Revision: 71210
>
> URL: http://llvm.org/viewvc/llvm-project?rev=71210&view=rev
> Log:
> This transform requires valid TargetData info. Wrap it in 'if (TD)' in
> preparation for the day we use null TargetData when no target is specified.
>

Instead of adding an extra indent level, how about adding:

if (!TD) {
  return;
}

before the TargetData-dependent code?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090508/97456b50/attachment.html>


More information about the llvm-commits mailing list