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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 5 13:35:55 PDT 2017

Previous message: [PATCH] D38597: [PEI] Remove required properties and use 'if' instead of std::function
Next message: [PATCH] D38597: [PEI] Remove required properties and use 'if' instead of std::function
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

MatzeB added a comment.

+1. You could still check the condition in an `assert()` at the right place though.


https://reviews.llvm.org/D38597

Previous message: [PATCH] D38597: [PEI] Remove required properties and use 'if' instead of std::function
Next message: [PATCH] D38597: [PEI] Remove required properties and use 'if' instead of std::function
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list