MaskRay added subscribers: Bdragon28, jhibbits, dim.
MaskRay added a comment.

In D137511#3911052 <https://reviews.llvm.org/D137511#3911052>, @thesamesam 
wrote:

> This needs to go in Breaking Changes in the release notes, not least so we 
> can link to it when updating upstreams.
>
> What do you mean by "while the darwin support has been removed from 
> llvm-project."? I don't think that's the case, if you mean that LLVM.org's 
> LLVM lacks support for Darwin.

Clarified it to "The darwin ppc support has been removed from llvm-project."

In D137511#3910880 <https://reviews.llvm.org/D137511#3910880>, @q66 wrote:

> The `__ppc__` macro should get the same treatment. That said, I believe there 
> are instances of both macros being used across the LLVM codebase, and those 
> cases are valid, so that should also be updated (one instance i know of is 
> the limits header of libcxx, there are likely more)

`__ppc__` is defined for FreeBSD in GCC, so perhaps we can make a separate 
change making `__ppc__` only defined for FreeBSD (@dim @Bdragon28 @jhibbits).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137511

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

Reply via email to