[llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp

[Anton Korobeynikov]

Jeff,

> Before assuming that the original code didn't work for Athlon64, the person who
> replaced it with a FIXME should have determined what did work.  Then he would have
> realized that the code was in fact correct, and would have avoided breaking it.
The prev. code didn't know about AMD processors at all. It was one big
"FIXME: support AMD processors". You can just look into diff for 1.53 to
verify it. I wasn't be able to verify 64-bit stuff that's why I put
another FIXME there. 

So, I don't see any breakage here.

-- 
With best regards, Anton Korobeynikov.

Faculty of Mathematics & Mechanics, Saint Petersburg State University.