[PATCH] D85389: Fix bug where we request a shadow memory one page larger than necessary.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 6 11:13:12 PDT 2020


tejohnson added a comment.

It looks reasonable, but adding @vitalybuka since this predates my recent refactoring, and he is probably a better person to review. It looks like the win code has the exact same issue, so should get a similar fix. The linux code doesn't seem to (it doesn't call FindAvailableMemoryRange, and doesn't have a similar comparison).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85389



More information about the llvm-commits mailing list