MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

Thank you for the indepth explaination in 
https://github.com/llvm/llvm-project/issues/59178, that was really helpful for 
me trying to understand what the problem was.

I thik your explaination is clear, the reason why you are suggesting this 
change, and as it passed all the test, hopefully we can apply the "Beyonce 
Rule" on  https://github.com/llvm/llvm-project/issues/56352, I recommend we 
pull in @curdeius  as a review

I'm going to accept this, but I would ask one of the others gives it the all 
clear too, let us know if you have commit access and if not and want one of us 
to land it we'll need you name and email to attribute the change to you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151047

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

Reply via email to