benedekaibas wrote: @steakhal @NagyDonat
Sorry for the ping, but I do not want to merge the PR without you knowing about [7071d00](https://github.com/llvm/llvm-project/pull/236804/commits/7071d00aaa5196b0b35caf224a3b584e05ad9894). During my review in the test files I have seen that https://github.com/llvm/llvm-project/blob/c750212b8c7c0cdc414dde893e302dbdcbd6776d/clang/test/Analysis/use-after-move.cpp#L583 uses an upper case 'E' which would be converted to a lowercase without implementing [7071d00](https://github.com/llvm/llvm-project/pull/236804/commits/7071d00aaa5196b0b35caf224a3b584e05ad9894). To avoid this regression I have written a basic if guard that only uses the lowercase sentence when the `os` is _not_ empty. https://github.com/llvm/llvm-project/pull/236804 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
