[PATCH] D153228: [clang-format] Fixed bad performance with enabled qualifier fixer.

Sedenion via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 19 11:34:42 PDT 2023


Sedeniono updated this revision to Diff 532721.
Sedeniono added a comment.

As suggested by HazardyKnusperkeks, increased the number of stack elements and removed the reserve(). That causes all Passes to be stored on the stack. (I counted 17 distinct possible passes, but not all of them can be active at the same time: Some are javascript and some a C++ exclusive.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153228

Files:
  clang/lib/Format/Format.cpp
  clang/lib/Format/QualifierAlignmentFixer.cpp
  clang/lib/Format/QualifierAlignmentFixer.h
  clang/unittests/Format/QualifierFixerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153228.532721.patch
Type: text/x-patch
Size: 8633 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230619/5225515c/attachment.bin>


More information about the cfe-commits mailing list