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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to