Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove deprecated AnalyzeTemporaryDtors #62020

Closed
carlosgalvezp opened this issue Apr 8, 2023 · 4 comments
Closed

Remove deprecated AnalyzeTemporaryDtors #62020

carlosgalvezp opened this issue Apr 8, 2023 · 4 comments
Assignees
@carlosgalvezp
Labels
clang-tidy code-cleanup
Milestone
LLVM 18.X Release

Comments

@carlosgalvezp
Copy link
Contributor

Deprecated in clang-tidy 16, fully remove in clang-tidy 18.
@carlosgalvezp carlosgalvezp added this to the LLVM 18.0.0 Release milestone Apr 8, 2023
@llvmbot
Copy link
Member

llvmbot commented Apr 8, 2023

@llvm/issue-subscribers-clang-tidy

@carlosgalvezp carlosgalvezp self-assigned this Jul 26, 2023
@carlosgalvezp carlosgalvezp added the awaiting-review Has pending Phabricator review label Jul 26, 2023
@carlosgalvezp
Copy link
Contributor Author

https://reviews.llvm.org/D156303

eymay pushed a commit to eymay/llvm-project that referenced this issue Jul 28, 2023
@carlosgalvezp @eymay
[clang-tidy] Remove AnalyzeTemporaryDestructors configuration option
bacd804 
Since it was deprecated since clang-tidy 16.

Fixes llvm#62020

Differential Revision: https://reviews.llvm.org/D156303
@PiotrZSL PiotrZSL removed the awaiting-review Has pending Phabricator review label Aug 6, 2023
rjoomen added a commit to tesseract-robotics/tesseract that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
e8ebc72 
See llvm/llvm-project#62020
@rjoomen rjoomen mentioned this issue Jan 8, 2024
Closed
rjoomen added a commit to tesseract-robotics/tesseract_planning that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
46ae100 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/trajopt that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
76a8291 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/tesseract_qt that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
42680b5 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/tesseract_ros2 that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
ea7ae8f 
See llvm/llvm-project#62020
This was referenced Jan 8, 2024
Merged
Merged
Merged
Merged
Merged
Levi-Armstrong pushed a commit to tesseract-robotics/trajopt that referenced this issue Jan 8, 2024
@rjoomen @Levi-Armstrong
Remove deprecated AnalyzeTemporaryDtors
0733775 
See llvm/llvm-project#62020
Levi-Armstrong pushed a commit to tesseract-robotics/tesseract_qt that referenced this issue Jan 8, 2024
@rjoomen @Levi-Armstrong
Remove deprecated AnalyzeTemporaryDtors
8042e6c 
See llvm/llvm-project#62020
Levi-Armstrong pushed a commit to Jmeyer1292/opw_kinematics that referenced this issue Jan 8, 2024
@rjoomen @Levi-Armstrong
Remove deprecated AnalyzeTemporaryDtors
80ed38d 
See llvm/llvm-project#62020
Levi-Armstrong pushed a commit to tesseract-robotics/tesseract_planning that referenced this issue Jan 8, 2024
@rjoomen @Levi-Armstrong
Remove deprecated AnalyzeTemporaryDtors
2e1e31c 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/tesseract_ros2 that referenced this issue Jan 8, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
a6440ac 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/tesseract that referenced this issue Jan 9, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors
f4ff46c 
See llvm/llvm-project#62020
rjoomen added a commit to tesseract-robotics/tesseract that referenced this issue Jan 9, 2024
@rjoomen
Remove deprecated AnalyzeTemporaryDtors (attempt 2)
5440e57 
See llvm/llvm-project#62020 and https://stackoverflow.com/questions/63920634/what-is-the-meaning-of-the-analyzetemporarydtors-option-in-clang-tidy
@rjoomen rjoomen mentioned this issue Jan 9, 2024
Closed
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Feb 6, 2024
@tstellar
Copy link
Collaborator

tstellar commented Feb 7, 2024

Is there a patch that needs to be backported for this?

@carlosgalvezp
Copy link
Contributor Author

Not that I'm aware of, why?

@tstellar tstellar moved this from Needs Triage to Done in LLVM Release Status Feb 16, 2024
@rockwotj rockwotj mentioned this issue Mar 20, 2024
Merged
6 tasks
@smuzaffar smuzaffar mentioned this issue May 23, 2024
Merged
@mojomex mojomex mentioned this issue Jul 10, 2024
Merged
4 tasks
wolfpld added a commit to wolfpld/tracy that referenced this issue Sep 22, 2024
@wolfpld
Remove AnalyzeTemporaryDtors clang-tidy setting.
07a5611 
llvm/llvm-project#62020
suoyuan666 added a commit to suoyuan666/ReleaseButler that referenced this issue Nov 13, 2024
@suoyuan666
fix .clang-tidy config error
9cc90f4 
AnalyzeTemporaryDtors be removed for clang-18
llvm/llvm-project#62020
@MichaelKutzner MichaelKutzner mentioned this issue Dec 17, 2024
Closed
@huangminghuang huangminghuang mentioned this issue Dec 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlosgalvezp carlosgalvezp

Labels
clang-tidy code-cleanup
Projects
LLVM Release Status
Status: Done
Milestone
LLVM 18.X Release
Development

No branches or pull requests
4 participants
@tstellar @PiotrZSL @carlosgalvezp @llvmbot