[clang] [clang][AMDGPU] Clean-up handling of named barrier type (PR #207687)

[Matt Arsenault via cfe-commits]

================
@@ -14263,6 +14263,17 @@ def note_acc_reduction_combiner_forming
     : Note<"while forming %select{|binary operator '%1'|conditional "
            "operator|final assignment operator}0">;
 
+// AMDGCN type diagnostics
+def err_amdgcn_invalid_field_not_a_wrapper : Error<
+  "%0 is only allowed as a field if the %select{struct|interface|union|class|enum}1"
----------------
arsenm wrote:

> Do you mean disallowing it in fields entirely ?

Yes. 

Either this relaxation should be applied to *all* opaque types, or none. I don't see why this one should get special treatment. 


https://github.com/llvm/llvm-project/pull/207687