[PATCH] D140741: [llvm-profdata] Remove unnecessary file size check

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 28 15:24:14 PST 2022


arsenm added a comment.

In D140741#4018895 <https://reviews.llvm.org/D140741#4018895>, @snehasish wrote:

> Some filesystems have a 4GB limit on file size (notably FAT32). Given that llvm-profdata as a tool is built for multiple platforms, I don't think we should remove this check.

Let the file system check that, don't force artificial restrictions in random places


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140741/new/

https://reviews.llvm.org/D140741



More information about the llvm-commits mailing list