<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 5, 2016 at 2:27 PM Simon Pilgrim via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">RKSimon added a subscriber: RKSimon.<br>
<br>
================<br>
Comment at: lib/Target/X86/X86ISelLowering.cpp:96<br>
@@ -96,1 +95,3 @@<br>
+    else if (Subtarget.has486Insns())<br>
+      setMaxAtomicSizeInBitsSupported(32); // has cmpxchg<br>
   }<br>
----------------<br>
Subtarget.has486Insns() and Subtarget.has586Insns() are quiet vague about actual features, would it not be clearer to instead add Subtarget.hasCMPXCHG8() and Subtarget.hasCMPXCHG() that then uses Has586/Has486 internally?<br>
<br>
Same for BSWAP (Subtarget.hasBSWAP()) below.<br><br></blockquote><div><br></div><div>Yes, this.</div><div><br></div><div>Subtarget features should be as specific as possible if you can. </div><div><br></div><div>-eric </div></div></div>