[all-commits] [llvm/llvm-project] 65053f: [NFC] Use more appropriate SmallVectorImpl::append...

yurai007 via All-commits all-commits at lists.llvm.org
Sat Jul 30 01:14:01 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 65053fbc0d461841db176f1143f3cc1d16997df0
      https://github.com/llvm/llvm-project/commit/65053fbc0d461841db176f1143f3cc1d16997df0
  Author: Dawid Jurczak <dawid_jurek at vp.pl>
  Date:   2022-07-30 (Sat, 30 Jul 2022)

  Changed paths:
    M llvm/include/llvm/ADT/SmallVector.h

  Log Message:
  -----------
  [NFC] Use more appropriate SmallVectorImpl::append call in std::initializer_list SmallVector constructor

Since we are in constructor there is no need to perform redundant call to SmallVectorImpl::clear() inside assign function.
Although calling cheaper append function instead assign doesn't make any difference on optimized builds
(DSE does the job removing stores), we still save some cycles for debug binaries.

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




More information about the All-commits mailing list