Skip to content

Latest commit

 

History

History
178 lines (133 loc) · 9.5 KB

0078-target-features.md

File metadata and controls

178 lines (133 loc) · 9.5 KB

Summary

Provide a standard and easily testable way to inspect features of a given target and provide them to the various parts of TVM which utilise that information.

Motivation

TVM has multiple ways to define a Targets architectural features for use in deciding on schedules or other calculations, here's a few different ways we do this:

This RFC aims to standardise the way in which we convert Target attributes into architectural features by processing them ahead of time.

Guide-level explanation

An additional property features will be added to the Target which is created at the time of instantiation, this will be populated by inferred features of the Target such as architectural extensions or bus sizes. The main distinction is that features are inferred from the Target attrs rather than being passed in.

An example of the new features attribute will be illustrated using examples targeting TVM for Arm(R) Cortex(R)-M4.

The Target specifies the specific CPU in the attrs and uses that to create the features object representing the architectural extensions of the Target, which can then be accessed using the GetFeature method similar to GetAttr:

Target my_target("c -mcpu=cortex-m4");
my_target->GetFeature<Bool>("is_aarch64", false); // false
my_target->GetFeature<Bool>("has_dsp", false); // true
my_target = Target("c -mcpu=cortex-m4")
my_target.features.is_aarch64 # false
my_target.features.has_dsp # true

This means that instead of the current:

isa = arm_isa.IsaAnalyzer(target)
if isa.has_dsp_support:
    do_dsp_stuff()

The Target can be directly inspected:

if target.features.dsp:
    do_dsp_stuff()

Reference-level explanation

The Target class, in C++, will have an an additional property named features:

class Target {
    ...
    DictAttrs features;
    
    ...
}

Which will have similar helper methods to those seen in IRModule for DictAttrs but with reference to Features rather than Attr:

template <typename TObjectRef>
Optional<TObjectRef> GetFeatures(
    const std::string& attr_key,
    Optional<TObjectRef> default_value = Optional<TObjectRef>(nullptr)) const {
    return attrs.GetAttr(attr_key, default_value);
}

template <typename TObjectRef>
Optional<TObjectRef> GetFeatures(const std::string& attr_key, TObjectRef default_value) const {
    return GetFeatures<TObjectRef>(attr_key, Optional<TObjectRef>(default_value));
}

As well as a Python class to represent this and allow simple access to the features using the target.features.<feature> syntax:

class TargetFeatures:
    def __init__(self, target):
        self._target = target

    def __getattr__(self, name):
        return _ffi_api.TargetGetFeature(self._target, name)

Drawbacks

Centralising features on Target increases the complexity for each Target parser as they will have to cater for a number of attributes, this is easily avoided by splitting the internal parsers.

Making features read-only and derived from the parser limits the flexibility to create an object with specific features for testing, in this case actual valid Targets will have to be used for such testing.

Rationale and alternatives

Re-use Target Attributes

If we were to attach all of these directly to Target (i.e. llvm) as attrs, that would drastically increase the number of fields on a given Target and in all cases only a subset would be used - specific to a given CPU/GPU profile:

my_target = Target("c -mcpu=cortex-m4")
my_target.is_aarch64 # Extra attribute in `attrs`

Re-using attrs becomes confusing to work with alongside the documented Target attributes in target_kind.cc, or target_kind.cc would need to be bloated with every potential feature of a Target. The approach of overlapping with Target attributes would also increase testing overhead rather than having a straight forward attrs to features map to test you would need to consider which attrs could validly mutate - this also introduces user confusion as target.mcpu is no longer the mcpu which they passed in.

Extend Utility Functions

Using a standalone function or class across the various areas of the codebase, such as:

TargetFeatures my_target_features(target)
my_target_features->is_aarch64; // false

This means re-processing Target whenever a specific attribute is required but would provide a single source of truth for doing so.

Target Tags

It's potentially possible to recreate the functionality of features by populating a larger list of Target tags, taking the example of:

TVM_REGISTER_TARGET_TAG("raspberry-pi/4b-aarch64")
    .set_config({{"kind", String("llvm")},
                 {"mtriple", String("aarch64-linux-gnu")},
                 {"mcpu", String("cortex-a72")},
                 {"mattr", Array<String>{"+neon"}},
                 {"num-cores", Integer(4)},
                 {"host", Map<String, ObjectRef>{{"kind", String("llvm")},
                                                 {"mtriple", String("aarch64-linux-gnu")},
                                                 {"mcpu", String("cortex-a72")},
                                                 {"mattr", Array<String>{"+neon"}},
                                                 {"num-cores", Integer(4)}}}});

These are pre-configured Targets with various mtriple, mcpu and mattr attributes already set - once parsed these can produce a set of architecture features for subsequent steps, such as replacing this check in the operator strategy:

https://github.com/apache/tvm/blob/f88a10fb00419c51a116a63f931a98d8286b23de/python/tvm/relay/op/strategy/arm_cpu.py#L232-L245

Other tagged Targets will likely have the same mattr and mcpu, thus rather than trying to hand craft the permutations each time, the parser generalises inferring these features, augmenting tagged Targets.

Prior art

Other Compilers

Taking the example of LLVM, it follows a similar methodology, resulting in a Features vector:

You can see similar definitions within GCC:

Existing TVM RFCs

This RFC builds upon the following existing TVM RFCs:

Unresolved questions

Future possibilities

Similar to LLVM and GCC, we may be able to use a custom file format to describe Targets more effectively in future which can be added using the same hooks, allowing for easier contributions.