[PATCH] D61281: [clang-format] Fixed self assignment

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 2 09:11:05 PDT 2019


RKSimon added a reviewer: RKSimon.
RKSimon added a comment.

In D61281#1485833 <https://reviews.llvm.org/D61281#1485833>, @MyDeveloperDay wrote:

> Did this cause some issue? Does this fix something if so can we add a test, because maybe the line isn't needed
>
> I would think we'd want to keep this as an identifier. we are just treating arg? the same as we would arg


@MyDeveloperDay didn't you write this code in D58404 <https://reviews.llvm.org/D58404>?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61281/new/

https://reviews.llvm.org/D61281





More information about the cfe-commits mailing list