efriedma added a comment.

The LLVM backend currently assumes every CPU is Pentium-compatible.  If we're 
going to change that in clang, we should probably fix the backend as well.



================
Comment at: lib/Basic/Targets/X86.h:472
+    CPUKind Kind = getCPUKind(Opts.CPU);
+    if (Kind >= CK_i586 || Kind == CK_Generic)
+      MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 64;
----------------
What exactly does "CK_Generic" mean here?  Under what circumstances does it 
show up? We should have a specific default for every supported target, if user 
doesn't specify a CPU with -march.


Repository:
  rC Clang

https://reviews.llvm.org/D42154



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to