[PATCH] Add a new matcher m_ICmpWithPred
Philip Reames
listmail at philipreames.com
Mon Dec 29 14:30:00 PST 2014
LGTM
Refactoring some bit of code as part of the patch so that this isn't dead on submission would be a good idea. There should be lots of candidates.
================
Comment at: include/llvm/IR/PatternMatch.h:738
@@ -719,1 +737,3 @@
+template<ICmpInst::Predicate P, typename LHS, typename RHS>
+inline CmpClassPred_match<LHS, RHS, ICmpInst, ICmpInst::Predicate, P>
----------------
Add a comment which clarifies the difference with the previous, it's not obvious.
Something along the lines of:
/// Matches an arbitrary ICmp and binds Pred
.. matcher 1
/// Matches an icmp w/ a specific predicate P
... matcher 2
http://reviews.llvm.org/D6692
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list