[PATCH] D137820: [MemoryBuffer] Allow optionally specifying desired buffer alignment

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 01:59:40 PST 2022


rriddle added a comment.

@dblaikie Would love thoughts if you think this is the right approach. I hit some interesting situations downstream where a small file with large alignment constraints(in the realm of 64/128/etc bytes) was seemingly miscompiling, but then I learned that MemoryBuffer only mmaps large files. This patch explicitly encodes alignment constraints, which felt safer than just changing the alignment in WritableMemoryBuffer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137820



More information about the llvm-commits mailing list