hans added a comment.

>>> Do you have a suggestion how we can move this patch forward?
>>
>> IIRC, D150688 <https://reviews.llvm.org/D150688> + the diff in 
>> https://github.com/llvm/llvm-project/issues/62719#issuecomment-1552903385 + 
>> upgrading the pre-merge linux bot should take care of all known issues.
>
> Would it make sense to put all these patches in one new review and then test 
> that on Chromium and ask @glandium to test that too. Then we know whether it 
> solves the issues. Do you want me to make a patch or do you want to do it?

I tested 
https://github.com/llvm/llvm-project/issues/62719#issuecomment-1552903385 on 
Chromium already, and that looks good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144509

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D144509: [CMake] Bum... Hans Wennborg via Phabricator via cfe-commits

Reply via email to