[PATCH] D39858: [PowerPC][msan] Update msan to handle changed memory layouts in newer kernels

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 06:05:31 PST 2017


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

LGTM.
I am far from an expert on this, but the modification seems to make perfect sense. Perhaps @eugenis can have a look as well, but if not, I'd say that we should go ahead with this since:

- It seems to make perfect sense (and shouldn't affect 46 and 45 bit VMA kernels)
- You've tested it on multiple kernels
- It's been up for review for around two weeks
- This is blocking us from running on 47 bit VMA kernels


https://reviews.llvm.org/D39858





More information about the llvm-commits mailing list