Skip to content

Fix compile error on Mac Clang 12.0.5 (#2058)#2069

Merged
zanmato1984 merged 1 commit intopingcap:release-4.0from
ti-srebot:release-4.0-93b548275fa5
Jun 3, 2021
Merged

Fix compile error on Mac Clang 12.0.5 (#2058)#2069
zanmato1984 merged 1 commit intopingcap:release-4.0from
ti-srebot:release-4.0-93b548275fa5

Conversation

@ti-srebot
Copy link
Collaborator

@ti-srebot ti-srebot commented Jun 3, 2021

cherry-pick #2058 to release-4.0
You can switch your code base to this Pull Request by using git-extras:

# In tics repo:
git pr https://github.com/pingcap/tics/pull/2069

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/tics.git pr/2069:release-4.0-93b548275fa5

What problem does this PR solve?

Issue Number: close #1887

Problem Summary:

There are a lot of changes since Mac Clang 12.0.5 comparing to 12.0.0. These changes bring up unintended compiler warning and errors.

To see some of what's changed in Mac Clang 12.0.5, please try this little program: https://github.com/zanmato1984/clang-1205-test

What is changed and how it works?

  • Regulate some codes.

  • Ignore some warnings.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added CHERRY-PICK cherry pick make-local-great-again Issues and fixes about building on MacOS Clang status/LGT1 Indicates that a PR has LGTM 1. labels Jun 3, 2021
@ti-srebot ti-srebot requested a review from JaySon-Huang June 3, 2021 10:18
@ti-srebot ti-srebot added this to the v4.0.14 milestone Jun 3, 2021
Copy link
Contributor

@zanmato1984 zanmato1984 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zanmato1984
Copy link
Contributor

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jun 3, 2021
@ti-srebot
Copy link
Collaborator Author

/run-all-tests

@zanmato1984 zanmato1984 merged commit 9df1558 into pingcap:release-4.0 Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CHERRY-PICK cherry pick make-local-great-again Issues and fixes about building on MacOS Clang status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants