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

Reply via email to