steakhal wrote:

> 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>`".

Could you edit the PR description and title as you see fit? @NagyDonat

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