[PATCH] CFE Knob for: Add a thread-model knob for lowering atomics on baremetal & single threaded systems

Tim Northover t.p.northover at gmail.com
Wed Sep 3 08:16:45 PDT 2014


================
Comment at: lib/CodeGen/BackendUtil.cpp:431
@@ +430,3 @@
+      .Case("posix", llvm::ThreadModel::POSIX)
+      .Case("single", llvm::ThreadModel::Single);
+
----------------
rengolin wrote:
> Shouldn't this have a .Default()? If so, which value?
And is "posix" really the best term for the C++11 model? Windows has just as much history of threading, and it was all rather a bodge before the C++11 description came around.

http://reviews.llvm.org/D4985






More information about the cfe-commits mailing list