pengfei added inline comments.

================
Comment at: llvm/lib/Target/X86/X86ISelDAGToDAG.cpp:4675
     }
+    case Intrinsic::x86_tilestored64_internal: {
+      unsigned Opc = X86::PTILESTOREDV;
----------------
yubing wrote:
> It seems there should be a check here, according to line4575:
> 
> ```
> if (!Subtarget->hasAMXTILE())
>   break;
> ```
> 
> 
> 
> 
Is the check in 4575 is necessary?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87981

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to