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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D28213: [Frontend] C... Ryan Prichard via Phabricator via cfe-commits

Reply via email to