[PATCH] Remove LowerInvoke's obsolete "-enable-correct-eh-support" option
Rafael Espíndola
rafael.espindola at gmail.com
Wed Mar 26 14:17:55 PDT 2014
On 26 March 2014 17:11, Mark Seaborn <mseaborn at chromium.org> wrote:
> 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).
Both names look good to me.
Cheers,
Rafael
More information about the llvm-commits
mailing list