[PATCH] D80497: Make mlir::Value's bool conversion operator explicit

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 01:35:15 PDT 2020


mehdi_amini added a comment.

In D80497#2055817 <https://reviews.llvm.org/D80497#2055817>, @dblaikie wrote:

> @mehdi_amini - would you mind writing some kind of comment when you approve a patch in Phabricator? Without any text in the approval, the email is not sent to the mailing list (only to the people in the review) & it looks like the patch is then committed without approval (according to the mail on the mailing list).


I don't think I can promise to remember this, this is too easy to forget to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80497





More information about the llvm-commits mailing list