aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Arsenault <Matthew.Arsenault@amd.com>2020-08-19 13:59:11 -0400
committerMatt Arsenault <Matthew.Arsenault@amd.com>2020-08-19 14:15:38 -0400
commit9e8d59a9b81ec152e33881ed5bd3aa07ac805db0 (patch)
tree002dc13e765a394b7452eb95a878ccdc7c53a5df
parenta208ad5ddb5bbcf773dae82d37755dc109dab483 (diff)
downloadllvm-9e8d59a9b81ec152e33881ed5bd3aa07ac805db0.zip
llvm-9e8d59a9b81ec152e33881ed5bd3aa07ac805db0.tar.gz
llvm-9e8d59a9b81ec152e33881ed5bd3aa07ac805db0.tar.bz2
AMDGPU/GlobalISel: Remove hack for combines forming illegal extloads
Previously we weren't adding the LegalizerInfo to the post-legalizer combiner. Since that's fixed, we don't need to try to filter out the one case that was breaking.
-rw-r--r--llvm/lib/Target/AMDGPU/AMDGPUCombine.td11
1 files changed, 1 insertions, 10 deletions
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUCombine.td b/llvm/lib/Target/AMDGPU/AMDGPUCombine.td
index faaf916..d243074 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUCombine.td
+++ b/llvm/lib/Target/AMDGPU/AMDGPUCombine.td
@@ -47,18 +47,9 @@ def AMDGPUPreLegalizerCombinerHelper: GICombinerHelper<
let DisableRuleOption = "amdgpuprelegalizercombiner-disable-rule";
}
-
-// FIXME: combines_for_extload can introduce illegal extloads which
-// aren't re-legalized.
-// FIXME: Is there a way to remove a single item from all_combines?
-def all_combines_minus_extload : GICombineGroup<[trivial_combines,
- ptr_add_immed_chain, combine_indexed_load_store, undef_combines,
- identity_combines]
->;
-
def AMDGPUPostLegalizerCombinerHelper: GICombinerHelper<
"AMDGPUGenPostLegalizerCombinerHelper",
- [all_combines_minus_extload, gfx6gfx7_combines,
+ [all_combines, gfx6gfx7_combines,
uchar_to_float, cvt_f32_ubyteN]> {
let DisableRuleOption = "amdgpupostlegalizercombiner-disable-rule";
}