[PATCH] D30025: [compiler-rt] [builtins] Fix building atomic.c with GCC

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 24 11:29:21 PST 2017


mgorny added a reviewer: doug.gregor.
mgorny added a subscriber: doug.gregor.
mgorny added a comment.

If I read the git correctly, the change that forbid defining builtins was initially made in https://reviews.llvm.org/rL64639. @doug.gregor, any chance you could help us over here? Is clang supposed to unconditionally reject those definitions, and are we supposed to always work-around it in compiler-rt, or should we consider adding some additional switch to allow them in clang?


Repository:
  rL LLVM

https://reviews.llvm.org/D30025





More information about the cfe-commits mailing list