xbolva00 added a comment.
In D147714#4249357 <https://reviews.llvm.org/D147714#4249357>, @efriedma wrote:
> An error message pointing at the call in the source code would be good
> enough, I think. Adding a "reason" might be nice, but not strictly necessary.
Good idea. So now for:
short i(void);
int j(void) {
[[clang::nonportable_musttail]]
return i();
}
Clang / LLVM on X86 emits:
> example.cpp:2:5: error: failed to perform tail call elimination on a call
> site marked musttail
> int j(void) {
>
> ^
With debug info:
> example.cpp:4:12: error: failed to perform tail call elimination on a call
> site marked musttail
>
> return i();
> ^
Nice improvement for end users.
So if no concerns, I will do necessary modifications for other backends.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147714/new/
https://reviews.llvm.org/D147714
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
