[PATCH] D42887: [Driver] Add option to manually control discarding value names in LLVM IR.

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: docs/UsersManual.rst:1859
+Controlling LLVM IR Output
+---------------------------
+
----------------
Underlining is incorrect here (too long).


================
Comment at: docs/UsersManual.rst:1861
+
+Controlling Values Names in LLVM IR
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
----------------
s/Values/Value


================
Comment at: docs/UsersManual.rst:1862
+Controlling Values Names in LLVM IR
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
----------------
Same here.


================
Comment at: docs/UsersManual.rst:1865
+Emitting value names in LLVM IR increases the size and verbosity of the IR.
+By default value names are only emitted in assertion enabled builds of Clang.
+However, when reading IR it can be useful to re-enable the emission of value
----------------
By default value -> By default, value
in assertion enabled builds -> in assertion-enabled builds


https://reviews.llvm.org/D42887