[PATCH] D81045: [MLIR] Modify HasParent trait to allow one of several op's as a parent

[Rahul Joshi via Phabricator via cfe-commits]

jurahul added inline comments.


================
Comment at: llvm/include/llvm/Support/Casting.h:147
+template <typename First, typename... Rest, typename Y>
+LLVM_NODISCARD inline typename std::enable_if<sizeof...(Rest) != 0, bool>::type
+isa(const Y &Val) {
----------------
jpienaar wrote:
> I'm worried that folks may not notice this change based on the change description (e.g., the change sounds MLIR specific), could you make this a separate change? (just focused on isa changes)
I did think of doing that. I agree that we should split the isa<> into its own change. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81045