[PATCH] D12937: [ARM] Handle +t2dsp feature as an ArchExtKind in ARMTargetParser.def
James Molloy via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 21 21:44:06 PDT 2015
I'd prefer to hear from Peter if he agrees with your course of action
first, as he raised concerns.
On Mon, 21 Sep 2015 at 13:54, A. Skrobov via llvm-commits <
llvm-commits at lists.llvm.org> wrote:
> tyomitch added a comment.
> Responding to Peter's comment above: the option 2 (adding FeatureDSPARM
> which can be enabled or disabled indepentently from FeatureDSPThumb2) seems
> a bit of overengineering for no gain: in the existing ARM architectures,
> the DSP extension is either available from all supported instruction sets,
> or from none of them.
> It would be ideal to call this "FeatureDSP", and to avoid associating this
> extension with either of the instruction sets; but unfortunately the
> "+t2dsp" syntax has already been exposed as part of LLVM/Clang user
> interface, and there may be external scripts specifying this feature name.
> Therefore, in my commit 248152 (later reverted by James) I went for option
> 1, that is, to update the description for FeatureDSPThumb2 to match its new
> Renato et al.: is it OK to recommit this patch, including its clang-side
> counterpart http://reviews.llvm.org/D12938 which James also reverted?
> rL LLVM
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits