[PATCH] D41866: AMDGPU: Error in SIAnnotateControlFlow instead of assert

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIAnnotateControlFlow.cpp:428
+    report_fatal_error("failed to annotate CFG");
+  }
+
----------------
cfang wrote:
> Does the annotation pass have the requirement of structurized CFG?  I remember that there is an OpenMP
> code with a conditional branch to a block with a unreachable instruction. Is that CFG unstructurized? 
Yes, the structurizer is a separate pass


https://reviews.llvm.org/D41866