[llvm] r273148 - [AARCH64] Add support for Broadcom Vulcan

[Renato Golin via llvm-commits]

On 20 June 2016 at 23:25, Eric Christopher <echristo at gmail.com> wrote:
> That said, one way I've seen these sorts of things being used is via
> ST->isCPUX() for TTI and a few other things which is fairly valid. We might
> need a better way of doing that, basically defining the name of the cpu as a
> subtarget feature is a bit wonky really.

We're moving things from isCPUX to hasFeature to get rid of long
conditionals (isX || isY || isZ || ... ) and to make matters clear why
is it that way.

But that doesn't preclude us from getting rid of the feature name and
its bad usage. We just want to keep the feature-based code gen, that's
all.

Btw, there are patches coming on ARM to replace isCPUX with target
features from us in the next few days / weeks, so we may want to wait
until that's done to refactor the SubtargetFeature list.

cheers,
--renato