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

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 9 12:55:25 PST 2016


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





More information about the llvm-commits mailing list