[PATCH] D93376: [LangRef] Clarify the semantics of lifetime intrinsics
Nuno Lopes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 18 09:01:22 PST 2020
nlopes added a comment.
As Ralf mentioned, the ship has sailed. Alloca and lifetime intrinsics were implemented like this several years ago. They were a quick hack to save stack space after inlining. That's it, and their design reflects the goals at the time.
We simply want to document what is implemented. @jdoerfert you seem to want to change the implementation and/or the design, which is a separate discussion. I suggest we first document how LLVM works and then if you want to make changes you start a *separate* discussion on the things you want to change, why, and what's the upgrade path, etc. We can't change the semantics of either alloca or lifetime intrinsics without an automatic upgrade path as otherwise we would break all frontends out there.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93376/new/
https://reviews.llvm.org/D93376
More information about the llvm-commits
mailing list