[PATCH] D35577: Add -flookup-tables and -fno-lookup-tables flags

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 27 09:32:40 PDT 2017


hans added a comment.

In https://reviews.llvm.org/D35577#818956, @mcrosier wrote:

> In https://reviews.llvm.org/D35577#818936, @kparzysz wrote:
>
> > In https://reviews.llvm.org/D35577#817944, @echristo wrote:
> >
> > > "Should this just be part of the tuning for the hexagon backend and not options at all"
> >
> >
> >
> >
> >   I don't think we need separate options to control jump tables and lookup tables: one option to enable/disable the use of memory tables would be sufficient.
>
>
> I don't have an objection to this direction (i.e., disable the lookup-table optimization when -fno-jump-tables is specified), so long as other are in agreement.


I'm ok with disabling lookup tables under `-fno-jump-tables` as well.


https://reviews.llvm.org/D35577





More information about the cfe-commits mailing list