[PATCH] D55562: Atomics: support min/max orthogonally

[Al Grant via Phabricator via cfe-commits]

algrant added a comment.
Herald added a project: clang.

Some targets (e.g. AArch64) support 8-bit, 16-bit and 64-bit atomics. max/min in Clang 7.0 only supported 32-bit max/min, even though the other atomics supported multiple widths. Is the intention here to support all (four) offered widths, signed and unsigned, and if so, could this be reflected in the tests, which are still only testing 32-bit?


Repository:
  rC Clang

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

https://reviews.llvm.org/D55562