[PATCH] D123345: Treat `std::move`, `forward`, and `move_if_noexcept` as builtins.

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

In D123345#3463224 <https://reviews.llvm.org/D123345#3463224>, @daltenty wrote:

> In D123345#3460639 <https://reviews.llvm.org/D123345#3460639>, @Jake-Egan wrote:
>
>> Hi, unfortunately there's a build failure on AIX: https://lab.llvm.org/buildbot/#/builders/214/builds/779/steps/9/logs/stdio. Could you take a look?
>
> Hi, any feedback on the AIX break? Seems like this might be non-trivial to address (were on an older libc++ level), so perhaps this needs a revert till can be worked out.

Sorry, I somehow missed this. What you're seeing is a symptom of a libc++ issue (PR12704) that was fixed >10 years ago (rGbff1bfc6be0615ba3036a861fd27b75c96e3297c <https://reviews.llvm.org/rGbff1bfc6be0615ba3036a861fd27b75c96e3297c>). I don't think we have explicit rules on how old a version of libc++ we aim to support, but trying to use version 3.1 or earlier with trunk Clang seems at least a little unreasonable. That said, it's probably not too hard to work around this, so I will :) but I would encourage whoever is responsible for libc++ in AIX to look into an upgrade!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123345