[PATCH] Remove LowerInvoke's obsolete "-enable-correct-eh-support" option

Mark Seaborn mseaborn at chromium.org
Wed Mar 26 14:11:06 PDT 2014


On 20 March 2014 10:33, Rafael Ávila de Espíndola <
rafael.espindola at gmail.com> wrote:

>   LGTM. It might be a good idea to rename the pass, but that can be done
> in another patch.


Do you have any recommendations about what to rename LowerInvoke to?  I
agree that "lower" isn't a great name, since the pass is stripping out
information irreversibly rather than lowering a feature while preserving
behaviour.  How about "StripInvokes"?

If we rename this, it would make sense to rename "LowerAtomic" to
"StripAtomics" too, since that's another pass which strips out information
irreversibly (like StripDeadPrototypes and StripSymbols).

Cheers,
Mark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140326/b0a462f8/attachment.html>


More information about the llvm-commits mailing list