[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