[PATCH] D51849: [RegisterCoalescer] Avoid "Use not jointly dominated by defs" in removePartialRedundancy

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 13 02:23:12 PDT 2018


uabelho added inline comments.


================
Comment at: test/CodeGen/AMDGPU/regcoal-removepartial-redundancy-not-jointly-dominated.ll:3
+;
+; This test was causing a "Use not jointly dominated by defs" when
+; removePartialRedundancy in RegisterCoalescing.cpp was calling extendToIndices
----------------
I tried running the test case on trunk now and it doesn't trigger the verifier even without the fix in this patch so something must have changed so the "wanted" situation doesn't happen with this ll-input anymore.


Repository:
  rL LLVM

https://reviews.llvm.org/D51849





More information about the llvm-commits mailing list