[PATCH] D28213: [Frontend] Correct values of ATOMIC_*_LOCK_FREE to match builtin

Thu May 26 18:14:21 PDT 2022

rprichard added a comment.

@mgorny Do you mind if I commandeer this Phabricator revision (and also D29542 <https://reviews.llvm.org/D29542>, which I would abandon in favor of D59566 <https://reviews.llvm.org/D59566> which is already merged)? I need this CL to get libc++ tests passing on i686-linux after D119931 <https://reviews.llvm.org/D119931> was submitted.

I rebased the InitPreprocessor.cpp change and updated two tests (Sema/atomic-ops.c and Preprocessor/init-x86.c). The Preprocessor/cuda-types.cu test is also failing, but I haven't investigated that yet.

Aside: FreeBSD switched its default CPU from i486 to i686 (D83645 <https://reviews.llvm.org/D83645>), but Clang's NetBSD target is still on 486, so that doesn't have cmpxchg8b by default.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D28213