[PATCH] D109823: [GlobalOpt] Do not shrink global to bool for an unfavorable AS

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/test/Transforms/GlobalOpt/shrink-global-to-bool.ll:33-41
+; CHECK-LABEL: @test_lds_var(
+; CHECK:    store i32 10, i32 addrspace(3)* @lvar, align 4
+; CHECK:    [[LD:%.*]] = load i32, i32 addrspace(3)* @lvar, align 4
+; CHECK:    store volatile i32 [[LD]], i32* undef, align 4
+;
+; AMDGCN-LABEL: @test_lds_var(
+; AMDGCN:    store i32 10, i32 addrspace(3)* @lvar, align 4
----------------
tra wrote:
> I'm confused. I see no differences between AMDGCN and CHECK anywhere in the test file.  AFAICT, the test would work if I were to use CHECK in the test with `--mtriple=amdgcn-amd-amdhsa`. If both CHECK and AMDGCN are identical, why do we need two different check patterns?
> 
> Did I miss something?
You did not: https://reviews.llvm.org/D109823#inline-1046556


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109823