[PATCH] D124342: [X86][AMX] Report error when shapes are not pre-defined.

Phoebe Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 24 02:43:00 PDT 2022


pengfei added inline comments.


================
Comment at: llvm/lib/Target/X86/X86PreTileConfig.cpp:52
+  Context.emitError(ErrorMsg);
+  exit(1);
+}
----------------
Any benefit to use `exit`? I found we seldom use it in the code.


================
Comment at: llvm/test/CodeGen/X86/AMX/amx-error.ll:8
+entry:
+  ; CHECK: Failed to config tile register
+  %t0 = load i16, ptr @row, align 2
----------------
Doesn't it success without this patch? https://godbolt.org/z/qvKcE3bzK


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124342



More information about the llvm-commits mailing list