[llvm] [memprof] Remove MemProf format Version 1 (PR #117357)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 22 17:19:06 PST 2024


kazutakahirata wrote:

> Hi @kazutakahirata,
> 
> There is the same problem on [llvm-clang-x86_64-expensive-checks-ubuntu](https://lab.llvm.org/buildbot/#/builders/187) builder.
> 
> ```
> /home/buildbot/worker/as-builder-4/ramdisk/expensive-checks/llvm-project/llvm/lib/ProfileData/InstrProfReader.cpp: In member function ‘llvm::Error llvm::IndexedMemProfReader::deserialize(const unsigned char*, uint64_t)’:
> /home/buildbot/worker/as-builder-4/ramdisk/expensive-checks/llvm-project/llvm/lib/ProfileData/InstrProfReader.cpp:1355:27: error: ‘Version1’ is not a member of ‘llvm::memprof’; did you mean ‘Version3’?
>  1355 |   if (Version <= memprof::Version1)
>       |                           ^~~~~~~~
>       |                           Version3
> ```

Thank you for bringing this to my attention.  I'll fix that right away.

https://github.com/llvm/llvm-project/pull/117357


More information about the llvm-commits mailing list