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.rev...@gmail.com>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78938

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

Reply via email to