[PATCH] D90829: [GlobalISel][TableGen] fix seg fault for zero instruction

Gabriel Hjort Ã…kerlund via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 23:11:59 PST 2020


ehjogab added inline comments.


================
Comment at: llvm/test/TableGen/GlobalISelEmitter-zero-instr.td:6
+
+// CHECK: return MatchTable
+def : Pat<(zext (i16 (trunc i32:$src))),
----------------
bjope wrote:
> Nice that we no longer crash, but I don't know how relevant this check is. What about checking that we get the "warning: Skipped pattern: Dst pattern root isn't a known leaf" message instead?
That'd be nice, but can it be done? I'm not very familiar with FileCheck... 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90829



More information about the llvm-commits mailing list