[PATCH] Propagate -fno-unwind-tables to EHABI

Renato Golin renato.golin at linaro.org
Mon Mar 17 10:44:38 PDT 2014



================
Comment at: test/CodeGen/ARM/ehabi.ll:283
@@ -282,3 +282,3 @@
 
-define void @test4() nounwind {
+define void @test4() uwtable nounwind {
 entry:
----------------
Logan Chien wrote:
> Is this change of test case correct?  IIRC, .cantunwind shouldn't be emitted when the function has uwtable attribute.
> 
> I remembered that Rafael had argued for this **feature** in the mailing list.
You need to have unwind tables to emit .cantunwind, so I'm not sure what do you mean. Since this function is a leaf function (and it doesn't throw), it cannot possible unwind, but it still needs the tables. It's still possible to unwind it in debug/profiling modes, though.

Right now, all functions in x86_64 and ARM will have uwtable (unless -fno-unwind-tables), and all non-throwing functions will have nounwind, so that can't be a feature, at least not right now.


http://llvm-reviews.chandlerc.com/D3079



More information about the llvm-commits mailing list