[PATCH] D78938: Make LLVM build in C++20 mode

Barry Revzin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 15 16:24:25 PST 2020


BRevzin added a comment.

In D78938#2450915 <https://reviews.llvm.org/D78938#2450915>, @nlopes wrote:

> Thanks @lebedev.ri for the pointer!
> I started working on exactly the same thing as I was trying to link a C++20 project with LLVM.
> @BRevzin is there anything missing in this patch? Do you have commit access or do you need help to land this?

There are two comments that @dblaikie made that need to still be addressed (one about renaming parameters of a function from `Self` and `Other` to `LHS` and `RHS`, and one about changing the ADL-inhibition strategy from `(to_address)(x)` to `llvm::to_address(x)`), both of which combined should take about a minute to do and then however long to compile.

I've forgotten how to push changes and am kind of confused at the state of my current branch anyway, and it takes so long to do anything on my laptop that I'm more than happy to let you take over. I do not have commit access anyway.

Per @MaskRay, I'm Barry Revzin <barry.revzin at gmail.com>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78938



More information about the cfe-commits mailing list