[PATCH] D35338: Add the -fdestroy-globals flag

[George Burgess IV via Phabricator via cfe-commits]

george.burgess.iv added a comment.

Thanks for working on this!

One small drive-by nit for you. Same "no need to update the diff this very second" as vsk; I can't LGTM this with confidence.

(Also, in the future, please try to include context <https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface> with your patch. Doing so makes reviewing it on Phab much easier. :) )



================
Comment at: include/clang/Frontend/CodeGenOptions.def:47
                                      ///< aliases to base ctors when possible.
+CODEGENOPT(DestroyGlobals, 1, 1) ///< -fdestroy-globals
 CODEGENOPT(DataSections      , 1, 0) ///< Set when -fdata-sections is enabled.
----------------
Nit: Please align the `///` with the above/below ones.


https://reviews.llvm.org/D35338