sbc100 added a comment.

In D57874#1389981 <https://reviews.llvm.org/D57874#1389981>, @sunfish wrote:

> > - `-matomics` means `-mthread-model posix`
>
> The others sound reasonable, though this one seems a little surprising -- a 
> user might have -matomics enabled because they're targeting a VM that has 
> atomics, but still not want to use -mthread-model posix because their code 
> doesn't actually using threads.


If you look at what "-mthread-model posix" actually means in the codebase it 
basically means "don't lower away atomics ops".

Its only used by WebAssembly and ARMTargetMachine to select the 
`createLowerAtomicPass`.   Given that these are the only uses it seems like it 
should be removed or renamed.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57874/new/

https://reviews.llvm.org/D57874



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

Reply via email to