[all-commits] [llvm/llvm-project] 3988bd: [llvm][clang][bolt][NFC] Use llvm::less_first() wh...

[Balazs Benics via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3988bd13988aad72ec979beb2361e8738584926b
      https://github.com/llvm/llvm-project/commit/3988bd13988aad72ec979beb2361e8738584926b
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-05-27 (Fri, 27 May 2022)

  Changed paths:
    M bolt/lib/Passes/LongJmp.cpp
    M clang/lib/Frontend/FrontendAction.cpp
    M clang/lib/Sema/SemaDeclCXX.cpp
    M clang/lib/Sema/SemaStmtAsm.cpp
    M clang/lib/StaticAnalyzer/Frontend/AnalyzerHelpFlags.cpp
    M clang/unittests/Introspection/IntrospectionTest.cpp
    M llvm/include/llvm/ExecutionEngine/Orc/Core.h
    M llvm/lib/IR/Attributes.cpp
    M llvm/lib/ObjCopy/MachO/MachOWriter.cpp
    M llvm/tools/dsymutil/DebugMap.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceAttributes.cpp

  Log Message:
  -----------
  [llvm][clang][bolt][NFC] Use llvm::less_first() when applicable

One could reuse this functor instead of rolling out your own version.
There were a couple other cases where the code was similar, but not
quite the same, such as it might have an assertion in the lambda or other
constructs. Thus, I've not touched any of those, as it might change the
behavior in some way.

As per https://discourse.llvm.org/t/submitting-simple-nfc-patches/62640/3?u=steakhal
Chris Lattner
> LLVM intentionally has a “yes, you can apply common sense judgement to
> things” policy when it comes to code review. If you are doing mechanical
> patches (e.g. adopting less_first) that apply to the entire monorepo,
> then you don’t need everyone in the monorepo to sign off on it. Having
> some +1 validation from someone is useful, but you don’t need everyone
> whose code you touch to weigh in.

Differential Revision: https://reviews.llvm.org/D126068