[PATCH] D17946: Add a flag to the LLVMContext to disable name for Value other than GlobalValue

[Philip Reames via llvm-commits]

reames added a subscriber: reames.
reames added a comment.

I don't really care if this lands in it's current form, but I'm somewhat of the belief this is the wrong approach to the problem.

An alternate approach which would seem more natural:

- We introduce an invariant for the optimizer wherein we only introduce named instructions if the instruction being replaced was named.  Same with basic blocks.
- We add a utility pass which drops names from instructions and basic blocks.
- Clang optionally schedules the utility pass early in it's pipeline.


http://reviews.llvm.org/D17946