[PATCH] D36086: [globalisel][tablegen] Add support for ImmLeaf without SDNodeXForm
Diana Picus via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 22 19:09:13 PDT 2017
rovka accepted this revision.
rovka added a comment.
This revision is now accepted and ready to land.
Thanks for the updates, this looks really good now!
================
Comment at: include/llvm/CodeGen/GlobalISel/InstructionSelectorImpl.h:143
+ "Expected G_CONSTANT");
+ assert(Predicate > 0 /*GIPFP_Invalid*/ && "Expected a valid predicate");
+ int64_t Value = 0;
----------------
Supernit: Predicate != GIPFP_Invalid seems more direct and readable...
https://reviews.llvm.org/D36086
More information about the llvm-commits
mailing list