[PATCH] D38597: [PEI] Remove required properties and use 'if' instead of std::function

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 6 11:18:05 PDT 2017


rnk added a comment.

I know I've only waited a day for @qcolombet and that's not much time, but I'm going to land this because the requirements checking change actually doesn't change our current functionality. Before this change, UsesCalleeSaves is initialized to false, so PEI doesn't require NoVRegs. This will actually give us better runtime checking.


https://reviews.llvm.org/D38597





More information about the llvm-commits mailing list