[PATCH] D42154: Don't generate inline atomics for i386/i486

[Eli Friedman via Phabricator via cfe-commits]

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