Meinersbur added a comment.

In D76342#1941591 <https://reviews.llvm.org/D76342#1941591>, @JonChesterfield 
wrote:
> edit: actually you've already done the clang-format on trunk as I hoped, 
> phabricator mislead me. Apologies for the noise


The previous diff was against a branch with only the whitespace changes to get 
rid of the noise in the diff. It seems that the pre-merge check instead 
compared formatting to the master (probably Arcanist/Phabricator looked for the 
common ancestor of origin/master and my git branch). This let to weird 
formatting warnings that weren't even changes highlighted in this diff. They 
are gone now with clang-formatted origin/master. Fotunately.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76342



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to