[PATCH] D81678: Introduce partialinit attribute at call sites for stricter poison analysis
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 16 17:05:12 PDT 2020
efriedma added a comment.
In terms of the C++ API, we definitely want to provide an API phrased positively in terms of individual arguments, so transforms don't have to deal with inverted logic.
In terms of the actual internal memory representation, or textual IR, maybe we can be a bit more flexible. `args_frozen` might be a reasonable compromise; the memory overhead should be pretty minimal even if we're attaching it to a bunch of functions, and it should be mostly readable. But maybe a little awkward to explain the textual IR if it has both `arg_frozen` and `args_frozen`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81678/new/
https://reviews.llvm.org/D81678
More information about the cfe-commits
mailing list