andykaylor wrote:

> What do you mean by CIRGenModule changes? I just added missing feature test, 
> as `setWindowsItaniumDLLImport` is unconditionally invoked on the path for 
> personality function generation, but it is still not supported.

Yeah, sorry for the confusion. I didn't notice that was the only change in 
CIRGenModule.

> I believe CIRGenException changes were already tested, I just moved 
> personality funciton generation from LowerToLLVM.

The `clang/test/CIR/Lowering/eh-inflight.cir` test only covers the lowering 
from CIR case. There is no test that handles the generation of the personality 
attribute from C++ source.

https://github.com/llvm/llvm-project/pull/171001
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to