[PATCH] D147714: [Attr] Introduce [[clang::nonportable_musttail]] as less strict version of [[clang::musttail]]

Josh Haberman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 24 14:24:55 PDT 2023


haberman added a comment.

I think I misunderstood @yamt's comment in https://reviews.llvm.org/D147714#4446780. I take back what I wrote above.

I agree that `[[clang::nonportable_musttail]]` is a nicer semantic, and the restrictions around the existing `[[clang:musttail]]` don't seem to buy us very much, since they are not universal enough to give a true assurance.

If someone could land a change to both LLVM and Clang to change the existing attribute, I would have no objection. But I have no idea what is involved in landing a backend (LLVM) change of that magnitude, especially since it would touch all the arch-specific backends.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147714/new/

https://reviews.llvm.org/D147714



More information about the cfe-commits mailing list