yronglin wrote:

> This broke tests (hidden due to #223794 breaking more tests earlier). Looks 
> like #224380 #224389 updates the tests. @yronglin, I merged #224380 #224389 
> to try and unbreak bots. Please verify that the test updates are expected; 
> otherwise please revert them and your change.

@yebinchon @nico Thanks for the quick fix and mentions! The failures caused by 
semantic conflict between this PR and 
https://github.com/llvm/llvm-project/pull/219952, the fix looks good to me. 
Somehow I forgot to rebase this PR after 
https://github.com/llvm/llvm-project/pull/219952 merged, I apologize for the 
problems this has caused.

https://github.com/llvm/llvm-project/pull/219288
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to