[PATCH] D135415: [GlobalISel] Add commutative matchers for compares.

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 13:45:49 PDT 2022


paquette added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/MIPatternMatch.h:705
+template <typename Pred, typename LHS, typename RHS>
+inline CompareOp_match<Pred, LHS, RHS, TargetOpcode::G_FCMP, true>
+m_c_GFCmp(const Pred &P, const LHS &L, const RHS &R) {
----------------
tschuett wrote:
> You could  put `[[nodiscard]]` here.
Should we do this for all of the matchers?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135415/new/

https://reviews.llvm.org/D135415



More information about the llvm-commits mailing list