Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp VirtRegMap.cpp VirtRegMap.h

2007-03-30 Thread Chris Lattner
First cut trivial re-materialization support. Woot! unsigned reg = rep(mop.getReg()); mii-getOperand(i).setReg(reg); +// If the definition instruction is re-materializable, its spill +// weight is zero. LiveInterval RegInt

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp VirtRegMap.cpp VirtRegMap.h

2007-03-30 Thread Evan Cheng
On Mar 30, 2007, at 2:44 PM, Chris Lattner wrote: First cut trivial re-materialization support. Woot! unsigned reg = rep(mop.getReg()); mii-getOperand(i).setReg(reg); +// If the definition instruction is re- materializable, its spill +

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp VirtRegMap.cpp VirtRegMap.h

2007-03-30 Thread Chris Lattner
On Mar 30, 2007, at 3:03 PM, Evan Cheng wrote: unsigned reg = rep(mop.getReg()); mii-getOperand(i).setReg(reg); +// If the definition instruction is re- materializable, its spill +// weight is zero. LiveInterval RegInt =

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp VirtRegMap.cpp VirtRegMap.h

2007-03-30 Thread Evan Cheng
On Mar 30, 2007, at 3:09 PM, Chris Lattner wrote: On Mar 30, 2007, at 3:03 PM, Evan Cheng wrote: unsigned reg = rep(mop.getReg()); mii-getOperand(i).setReg(reg); +// If the definition instruction is re- materializable, its spill +//

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp VirtRegMap.cpp VirtRegMap.h

2007-03-30 Thread Chris Lattner
- (mop.isUse() + mop.isDef()) * pow(10.0F, (int) loopDepth); +if (!RegInt.remat) { + RegInt.weight += +(mop.isUse() + mop.isDef()) * pow(10.0F, (int) loopDepth); +} Should it really be zero? It should definitely be lower

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp

2006-11-08 Thread John Criswell
Dear Mr. Laskey, This change broke the LLVM build on OpenBSD. Would it be possible to use FLT_MAX instead of HUGE_VALF? FLT_MAX from float.h looks more portable. -- John T. Jim Laskey wrote: Changes in directory llvm/lib/CodeGen: LiveIntervalAnalysis.cpp updated: 1.190 - 1.191

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp

2006-11-08 Thread Jim Laskey
John,Both are defined on Darwin, FLT_MAX in float.h and HUGE_VALF in math.h.  However, neither header seems to be based on a BSD header.  I choose HUGE_VALF because it was also defined in the LINUX and FREEBSD math.h headers (seemed like confirmation to me.)  I'll wait for Chris to make the call.

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp

2006-11-08 Thread Jim Laskey
John,Adding declaration to DataTypes.h(.in).Cheers,-- JimOn Nov 8, 2006, at 12:30 PM, Jim Laskey wrote:John,Both are defined on Darwin, FLT_MAX in float.h and HUGE_VALF in math.h.  However, neither header seems to be based on a BSD header.  I choose HUGE_VALF because it was also defined in the

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp

2006-11-08 Thread Bill Wendling
On 11/8/06, Jim Laskey [EMAIL PROTECTED] wrote: John, Both are defined on Darwin, FLT_MAX in float.h and HUGE_VALF in math.h. However, neither header seems to be based on a BSD header. I choose HUGE_VALF because it was also defined in the LINUX and FREEBSD math.h headers (seemed like

Re: [llvm-commits] CVS: llvm/lib/CodeGen/LiveIntervalAnalysis.cpp RegAllocLinearScan.cpp

2006-11-08 Thread Chris Lattner
On Nov 8, 2006, at 9:24 AM, Bill Wendling wrote: On 11/8/06, Jim Laskey [EMAIL PROTECTED] wrote: John, Both are defined on Darwin, FLT_MAX in float.h and HUGE_VALF in math.h. However, neither header seems to be based on a BSD header. I choose HUGE_VALF because it was also defined in