Differential D127859

[clangd] Don't add inlay hints on std::move/forward
ClosedPublic
Actions

Authored by upsj on Jun 15 2022, 7:29 AM.

Tags

Subscribers

Details

Reviewers

Commits

rGe984e1cd6137: [clangd] Don't add inlay hints on std::move/forward

Summary

This removes parameter inlay hints from a few builtin functions like std::move/std::forward

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

upsj created this revision.Jun 15 2022, 7:29 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2022, 7:29 AM

Herald added subscribers: usaxena95, kadircet, arphaman. · View Herald Transcript

upsj requested review of this revision.Jun 15 2022, 7:29 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2022, 7:29 AM

Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. · View Herald Transcript

Harbormaster completed remote builds in B169989: Diff 437162.Jun 15 2022, 8:26 AM

upsj mentioned this in D124690: [clangd] add inlay hints for std::forward-ed parameter packs.Jun 15 2022, 8:44 AM

Thanks!

This revision is now accepted and ready to land.Jun 16 2022, 12:26 AM

Closed by commit rGe984e1cd6137: [clangd] Don't add inlay hints on std::move/forward (authored by upsj). · Explain WhyJun 16 2022, 3:43 AM

This revision was automatically updated to reflect the committed changes.

upsj added a commit: rGe984e1cd6137: [clangd] Don't add inlay hints on std::move/forward.

Revision Contents
Changeset List

Path

Size

clang-tools-extra/

clangd/

21 lines

unittests/

InlayHintTests.cpp

11 lines

Diff 437495

clang-tools-extra/clangd/InlayHints.cpp

Loading...

clang-tools-extra/clangd/unittests/InlayHintTests.cpp

Loading...