[PATCH] D130406: Use llvm::sort instead of std::sort where possible

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 22 18:21:44 PDT 2022


gribozavr created this revision.
Herald added subscribers: bzcheeseman, sdasgup3, carlosgalvezp, wenzhicui, wrengr, cota, teijeong, rdzhabarov, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mgester, arpith-jacob, antiagainst, shauheen, rriddle, mehdi_amini, usaxena95, kadircet, arphaman, mgrang, hiraditya, arichardson, emaste.
Herald added a reviewer: rriddle.
Herald added a reviewer: MaskRay.
Herald added a reviewer: bondhugula.
Herald added a project: All.
gribozavr requested review of this revision.
Herald added a reviewer: nicolasvasilache.
Herald added subscribers: cfe-commits, llvm-commits, StephenFan, stephenneuendorffer, nicolasvasilache.
Herald added projects: clang, MLIR, LLVM, clang-tools-extra.

llvm::sort is beneficial even when we use the iterator-based overload,
since it can optionally shuffle the elements (to detect
non-determinism). However llvm::sort is not usable everywhere, for
example, in compiler-rt.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D130406

Files:
  clang-tools-extra/clang-include-fixer/IncludeFixerContext.cpp
  clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
  clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp
  clang-tools-extra/clang-tidy/llvm/IncludeOrderCheck.cpp
  clang-tools-extra/clang-tidy/misc/ConfusableTable/BuildConfusableTable.cpp
  clang-tools-extra/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
  clang-tools-extra/clangd/IncludeFixer.cpp
  clang-tools-extra/pseudo/lib/grammar/LRTableBuild.cpp
  clang/lib/CodeGen/TargetInfo.cpp
  clang/unittests/Tooling/TransformerTest.cpp
  lld/COFF/Chunks.cpp
  lld/COFF/DLL.cpp
  lld/COFF/DriverUtils.cpp
  lld/ELF/SyntheticSections.cpp
  llvm/lib/Bitcode/Writer/ValueEnumerator.cpp
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
  llvm/lib/MC/MCPseudoProbe.cpp
  llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp
  llvm/lib/Target/DirectX/DXILWriter/DXILValueEnumerator.cpp
  llvm/lib/Target/XCore/XCoreFrameLowering.cpp
  llvm/unittests/ADT/SmallSetTest.cpp
  llvm/unittests/MIR/MachineMetadata.cpp
  llvm/unittests/Support/AlignmentTest.cpp
  llvm/utils/TableGen/DXILEmitter.cpp
  llvm/utils/TableGen/DirectiveEmitter.cpp
  llvm/utils/TableGen/SearchableTableEmitter.cpp
  llvm/utils/UnicodeData/UnicodeNameMappingGenerator.cpp
  mlir/lib/Analysis/Liveness.cpp
  mlir/lib/Dialect/Affine/Transforms/LoopFusion.cpp
  mlir/lib/Dialect/SCF/Utils/Utils.cpp
  mlir/lib/IR/AffineExpr.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130406.447019.patch
Type: text/x-patch
Size: 31749 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220723/1cfcd7b3/attachment-0001.bin>


More information about the cfe-commits mailing list