[PATCH] D33916: RenameIndependentSubregs: Fix handling of undef tied operands

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 5 15:37:05 PDT 2017


arsenm added inline comments.


================
Comment at: test/CodeGen/AMDGPU/rename-independent-subregs-invalid-mac-operands.mir:2-23
+--- |
+  define amdgpu_kernel void @mac_invalid_operands(float %arg) #0 {
+  bb:
+    %tmp = fcmp oeq float %arg, 0.000000e+00
+    br i1 %tmp, label %bb1, label %bb6
+
+  bb1:                                              ; preds = %bb
----------------
MatzeB wrote:
> Do you actually need the IR, see also: http://llvm.org/docs/MIRLangRef.html#simplifying-mir-files
It complains if I try to remove it entirely. It seems to want the blocks at least


https://reviews.llvm.org/D33916





More information about the llvm-commits mailing list