yonghong-song wrote:
Thanks @teresajohnson Just updated a new revision to address your comments:
- Rename DisableAlwaysRenamePromotedLocals back to
AlwaysRenamePromotedLocals.
- Add more description for new tests.
- Add comments to explain usage of ExternallyVisibleSymbolNamesPtr.
Regarding the force-push. Typically before pushing a new revision, I rebase on
top of 'main' branch so the pull request is close to the 'main' branch. But
unfortunately, once I did rebase, I have to do force-push. Otherwise git won't
allow me to push the pull request.
https://github.com/llvm/llvm-project/pull/178587
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits