[PATCH] D29542: [TargetInfo] Adjust x86-32 atomic support to the CPU used

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 6 00:19:30 PST 2017


mgorny added inline comments.


================
Comment at: lib/Basic/Targets.cpp:1808
+    if (HostTriple.getArch() == llvm::Triple::x86)
+      HostTarget->setCPU("i586");
+
----------------
jlebar wrote:
> Okay, is this still needed now?
Yes. I've specifically tested with it commented out, and the CPU gets initiated to generic (=no inline atomics) then.


https://reviews.llvm.org/D29542





More information about the cfe-commits mailing list