2011/9/29 Rafael Ávila de Espíndola <[email protected]>:
> The attached patch is the first clang one for PR11038. It makes clang
> propagate the attribute from C the LLVM IL. I still doesn't add it for
> functions known to return twice like setjmp.
>
> Is it OK?
+ if (D->hasAttr<ReturnsTwiceAttr>()) {
+ F->addFnAttr(llvm::Attribute::ReturnsTwice);
+ }
Braces not necessary here.
I think the change in lib/CodeGen/CodeGenModule.cpp is redundant.
Otherwise, fine.
-Eli
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
