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

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 8 10:30:24 PDT 2020


jfb accepted this revision.
jfb added a comment.
This revision is now accepted and ready to land.

Two minor corrections, looks good otherwise.



================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:491
+def err_drv_trivial_auto_var_init_stop_after_invalid_value : Error<
+  "-ftrivial-auto-var-init-stop-after=* only accpets positive integers.">;
+
----------------
Typo "accepts".


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:5973
+          DiagnosticsEngine::Warning,
+          "-ftrivial-auto-var-init-stop-after=* has been enabled to limit the "
+          "number of times ftrivial-auto-var-init=%0 gets applied.");
----------------
Could you print out the count here, instead of `*`?



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