[PATCH] D88409: [SVE] Make ElementCount and TypeSize use a new PolySize class
Christopher Tetreault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 7 09:53:20 PDT 2020
ctetreau added a comment.
In D88409#2314381 <https://reviews.llvm.org/D88409#2314381>, @david-arm wrote:
> Hi @ctetreau, I decided to remove the default constructors as we only need a one or two line change llvm/include/llvm/IR/Intrinsics.h. This removes the need for asserts that we have a properly constructed object. Does this seem reasonable?
I think that would be idea.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88409/new/
https://reviews.llvm.org/D88409
More information about the llvm-commits
mailing list