craig.topper added inline comments.
================ Comment at: clang/include/clang/Support/RISCVVIntrinsicUtils.h:131 + bool operator==(const Policy &Other) const { + return this->PolicyNone == Other.PolicyNone && this->TU == Other.TU && + this->TA == Other.TA && this->MU == Other.MU && ---------------- You can drop `this->` ================ Comment at: clang/include/clang/Support/RISCVVIntrinsicUtils.h:372 unsigned NF = 1; - Policy DefaultPolicy = Policy::PolicyNone; + Policy DefaultPolicy = Policy(); ---------------- `= Policy()` is unnecessary. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139995/new/ https://reviews.llvm.org/D139995 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits