[llvm-commits] [llvm] r60461 - in /llvm/trunk: include/llvm/Target/Target.td include/llvm/Target/TargetInstrDesc.h lib/Target/X86/X86InstrInfo.cpp lib/Target/X86/X86InstrSSE.td test/CodeGen/X86/fold-pcmpeqd-0.ll test/CodeGen/X86/fold-pcmpeqd-1.ll utils/TableGen/CodeGenDAGPatterns.cpp

[Dan Gohman]

On Dec 2, 2008, at 11:20 PM, Evan Cheng wrote:

> Hi Dan,
>
> You are changing the meaning of isSimpleLoad to something that's not
> at all obvious form the name. Perhaps we should add a new
> TargetInstrDesc flag (canChangeIntoSimpleLoad?).

Ok, I just renamed isSimpleLoad to canFoldAsLoad, which I believe
captures the new generalized meaning.

> BranchFolding also
> uses isSimpleLoad.

That's already fixed.

Dan