[PATCH] D77168: Add a flag to debug automatic variable initialization

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 17 17:31:24 PDT 2020


jfb added a comment.

Automatic narrowing of bugs is indeed compelling, so I'd support that as long as it:

- Allows bracketing as John suggested (lower / upper bounds where to stop / start).
- Is implemented in a way which makes it really hard to regress the security mitigation. Maybe this requires emitting a diagnostic when auto-init isn't applied because of the flag.
- Is thorough (i.e. it covers all auto-init, for example `initializeAlloca` was missing).

I'd also like to see the pragma attribute approach, as well as byte-pattern variability as I described. I don't think auto-narrowing is the only approach we should push people towards.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77168





More information about the cfe-commits mailing list