<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

</head>

<body bgcolor="#ffffff" text="#000000">

Looks like Evan broke it last October.<br>

<br>

Jeff Cohen wrote:

<blockquote cite="mid46242332.6020308@jolt-lang.org" type="cite">

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

Yes, it worked for AMD.  I have AMD, and I wrote the code to support

AMD.  I left the code the way it was because it was correct for AMD.  I

actually tested it on an AMD system.<br>

  <br>

Anton Korobeynikov wrote:

  <blockquote cite="mid1176761194.13159.65.camel@asl.dorms.spbu.ru"

 type="cite">

    <pre wrap="">Jeff,

  </pre>

    <blockquote type="cite">

      <pre wrap="">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.

    </pre>

    </blockquote>

    <pre wrap=""><!---->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.

  </pre>

  </blockquote>

  <pre wrap="">

<hr size="4" width="90%">

_______________________________________________

llvm-commits mailing list

<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>

<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a>

  </pre>

</blockquote>

</body>

</html>