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

[Jonathan Roelofs]

On 9/3/14, 9:04 AM, Renato Golin wrote:
> ================
> Comment at: lib/CodeGen/BackendUtil.cpp:431
> @@ +430,3 @@
> +      .Case("posix", llvm::ThreadModel::POSIX)
> +      .Case("single", llvm::ThreadModel::Single);
> +
> ----------------
> Shouldn't this have a .Default()? If so, which value?
 From earlier in the thread:

"I considered it, but Stringswitch wouldn't let me assert that every case was 
handled."

"StringSwitch does that automatically for you when getting the result
if you leave off the .Default call."


Jon
>
> http://reviews.llvm.org/D4985
>
>

-- 
Jon Roelofs
jonathan at codesourcery.com
CodeSourcery / Mentor Embedded