tuliom added a comment.

In D159115#4640388 <https://reviews.llvm.org/D159115#4640388>, @mgorny wrote:

> I'm sorry, by "new" you mean the version with `uintptr_t` or the original as 
> well?

Both with `uintptr_t` as well as `unsigned long long`.

In D159115#4641721 <https://reviews.llvm.org/D159115#4641721>, @mgorny wrote:

> I've tested right now (as of 3398744a6106c83993611bd3c5e79ec6b94417dc 
> <https://reviews.llvm.org/rG3398744a6106c83993611bd3c5e79ec6b94417dc>) and 
> all clang tests passed for me on Gentoo/ppc64le.

Did Gentoo switch the default long double to IEEE 128-bit floating point?
If not, that could explain why this new code is broken on ppc64le.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159115

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D... Michał Górny via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Sam James via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits

Reply via email to