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

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 08:37:17 PDT 2017


mcrosier added a comment.

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.


https://reviews.llvm.org/D35577





More information about the llvm-commits mailing list