[PATCH] D18762: Add Aggressive Control Dead Code Elimination

[David Callahan via llvm-commits]

david2050 added a comment.

From Chandler by email:

> t would be good to at least send a fresh mail to the list with an updated rationale and such.

> 

> One thing that I'm having trouble even making out from the patch description in Phab is exactly what numbers you have collected. The formatting makes it really hard to understand what change this actually introduces.

> 

> As one high-level comment, why would we want this *and* ADCE to exist? I feel like we don't need 4 or 5 different dead code elimination passes and would benefit from this actually just being a patch to one of the existing ones.

> 

> Another high-level comment: if we must have both ADCE and this pass, I find the name "ACDCE" or "Aggressive Control Dead Code Elimination" very confusing. The order of the words seems all wrong. If we need to distinguish between eliminating dead control flows independent of eliminating dead instructions within basic blocks, I would name them "Aggressive Dead Control Flow Elimination" and "Aggressive Dead Code Elimination". And I would consider stripping the "Aggressive" prefix because I doubt that really makes sense these days.


Thanks Chandler

Changes in response shortly.


http://reviews.llvm.org/D18762