https://github.com/NagyDonat approved this pull request.

The change LGTM, but at first I was very confused by the PR description, more 
precisely the phrasing "Fixing <PR LINK> for <PROBLEM DESCRIPTION>" in the 
first two bullet points. I'm not a native English speaker, so this may be a 
valid expression that I'm not familiar with, but I would suggest rephrasing 
these sentences as "Fixing <PROBLEM DESCRIPTION> introduced in <PR LINK>".

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

Reply via email to