[PATCH] D23584: AMDGPU: Don't required structured CFG

[Nicolai Hähnle via llvm-commits]

nhaehnle added a subscriber: nhaehnle.
nhaehnle added a comment.

This looks reasonable to me, also I think you should remove requiresStructuredCFG entirely with this change. It doesn't look like anybody else is using it.

Just one small comment below.


================
Comment at: test/CodeGen/AMDGPU/indirect-addressing-si.ll:576
@@ -571,2 +575,3 @@
 declare i32 @llvm.amdgcn.workitem.id.x() #1
+declare void @llvm.amdgcn.s.barrier() #2
 
----------------
This is unnecessary.


https://reviews.llvm.org/D23584