[llvm-commits] [llvm] r102941 - /llvm/trunk/lib/Target/X86/X86TargetMachine.cpp

[Eric Christopher]

On May 3, 2010, at 11:13 PM, Evan Cheng wrote:

> 
> On May 3, 2010, at 10:18 PM, Chris Lattner wrote:
> 
>> 
>> On May 3, 2010, at 10:09 PM, Eric Christopher wrote:
>> 
>>> 
>>> On May 3, 2010, at 9:51 PM, Chris Lattner wrote:
>>> 
>>>> curious, what's this for?
>>> 
>>> Oh, I wanted to turn it off to check if there was a bug so I added a quick option for it. Figured I'd just commit it too.
>> 
>> Ok, this seems more like something that should be a -mattr flag based on the CPU, not a random llc hook.
> 
> I don't think it's meant to be a permanent option, right? Eric, are you just toggling it for try to track down a bug?

I was yes, but I figured being able to turn it off might be of use for others to track down bugs.  Didn't think there was much overhead in adding a new option.  I can take it out if it's a problem.

-eric