[all-commits] [llvm/llvm-project] a7fb86: Revert "[llvm-profdata] Fix mixed-sign comparison ...

dyung via All-commits all-commits at lists.llvm.org
Fri Jun 23 17:59:17 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a7fb8659b2a418b0dafcb96337133e8b7fa1e03c
      https://github.com/llvm/llvm-project/commit/a7fb8659b2a418b0dafcb96337133e8b7fa1e03c
  Author: Douglas Yung <douglas.yung at sony.com>
  Date:   2023-06-23 (Fri, 23 Jun 2023)

  Changed paths:
    M llvm/unittests/tools/llvm-profdata/MD5CollisionTest.cpp

  Log Message:
  -----------
  Revert "[llvm-profdata] Fix mixed-sign comparison warnings"

This reverts commit a2e7f26160359ed3913badb5acdd05aa8c663b46.

The original patch (31af18b) will be reverted, so reverting this follow-up patch first.


  Commit: c9a8a0e8a9b267fae0352af352e9c735fef019f7
      https://github.com/llvm/llvm-project/commit/c9a8a0e8a9b267fae0352af352e9c735fef019f7
  Author: Douglas Yung <douglas.yung at sony.com>
  Date:   2023-06-23 (Fri, 23 Jun 2023)

  Changed paths:
    M llvm/include/llvm/ProfileData/SampleProf.h
    M llvm/include/llvm/ProfileData/SampleProfReader.h
    M llvm/lib/ProfileData/ProfileSummaryBuilder.cpp
    M llvm/lib/ProfileData/SampleProf.cpp
    M llvm/lib/ProfileData/SampleProfReader.cpp
    M llvm/lib/ProfileData/SampleProfWriter.cpp
    M llvm/lib/Transforms/IPO/SampleContextTracker.cpp
    M llvm/test/tools/llvm-profdata/Inputs/sample-nametable-after-samples.profdata
    M llvm/test/tools/llvm-profdata/sample-nametable.test
    M llvm/tools/llvm-profdata/llvm-profdata.cpp
    M llvm/tools/llvm-profgen/ProfileGenerator.cpp
    M llvm/unittests/tools/llvm-profdata/CMakeLists.txt
    R llvm/unittests/tools/llvm-profdata/MD5CollisionTest.cpp
    M llvm/unittests/tools/llvm-profdata/OutputSizeLimitTest.cpp

  Log Message:
  -----------
  Revert "[llvm-profdata] Refactoring Sample Profile Reader to increase FDO build speed using MD5 as key to Sample Profile map"

This reverts commit 31af18bccea95fe1ae8aa2c51cf7c8e92a1c208e.

This change is causing build failures on many Windows build bots:

https://lab.llvm.org/buildbot/#/builders/216/builds/22833
https://lab.llvm.org/buildbot/#/builders/123/builds/19602
https://lab.llvm.org/buildbot/#/builders/172/builds/28315
https://lab.llvm.org/buildbot/#/builders/119/builds/13870
https://lab.llvm.org/buildbot/#/builders/233/builds/794
https://lab.llvm.org/buildbot/#/builders/235/builds/387
https://lab.llvm.org/buildbot/#/builders/13/builds/36921
https://lab.llvm.org/buildbot/#/builders/127/builds/50510


Compare: https://github.com/llvm/llvm-project/compare/0bedb667af1d...c9a8a0e8a9b2


More information about the All-commits mailing list