[all-commits] [llvm/llvm-project] db810b: [libclang] Remove duplicate dependency on LLVMSupport

[MyDeveloperDay via All-commits]

  Branch: refs/heads/release/10.x
  Home:   https://github.com/llvm/llvm-project
  Commit: db810b69f805e078b73c23fcc98f271fb05265bc
      https://github.com/llvm/llvm-project/commit/db810b69f805e078b73c23fcc98f271fb05265bc
  Author: Jan Korous <jkorous at apple.com>
  Date:   2020-05-07 (Thu, 07 May 2020)

  Changed paths:
    M clang/tools/libclang/CMakeLists.txt

  Log Message:
  -----------
  [libclang] Remove duplicate dependency on LLVMSupport

Differential Revision: https://reviews.llvm.org/D79451

(cherry picked from commit 02b303321d3f0d3b2c69f68aa25560848dd61f98)


  Commit: 7ae6db9cf0c0b11a6f753cb1cf8dabb512e70f6f
      https://github.com/llvm/llvm-project/commit/7ae6db9cf0c0b11a6f753cb1cf8dabb512e70f6f
  Author: Hans Wennborg <hans at chromium.org>
  Date:   2020-05-07 (Thu, 07 May 2020)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  clang-format: Fix pointer alignment for overloaded operators (PR45107)

This fixes a regression from D69573 which broke the following example:

  $ echo 'operator C<T>*();' | bin/clang-format --style=Chromium
  operator C<T> *();

(There should be no space before the asterisk.)

It seems the problem is in TokenAnnotator::spaceRequiredBetween(),
which only looked at the token to the left of the * to see if it was a
type or not. That code only handled simple types or identifiers, not
templates or qualified types. This patch addresses that.

Differential revision: https://reviews.llvm.org/D76850

(cherry picked from commit eb85e90350e93a64279139e7eca9ca40c8fbf5eb)


  Commit: baeb500a8ca52b144d1bc14a04130964d68eaade
      https://github.com/llvm/llvm-project/commit/baeb500a8ca52b144d1bc14a04130964d68eaade
  Author: mydeveloperday <mydeveloperday at gmail.com>
  Date:   2020-05-07 (Thu, 07 May 2020)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  [clang-format] [PR45357] Fix issue found with operator spacing

Summary:
This is a tentative fix for https://bugs.llvm.org/show_bug.cgi?id=45357

Spaces seem to be introduced between * and * due to changes brought in for {D69573}

Reviewers: sylvestre.ledru, mitchell-stellar, sammccall, Abpostelnicu, krasimir, jbcoe

Reviewed By: Abpostelnicu

Subscribers: tstellar, hans, Abpostelnicu, cfe-commits

Tags: #clang, #clang-format

Differential Revision: https://reviews.llvm.org/D78879

(cherry picked from commit b01dca50085768f1f1a5ad21a685906d48c38816)


Compare: https://github.com/llvm/llvm-project/compare/bab8d1790a33...baeb500a8ca5