I think AdvancedIncSubtensor already inherits from Op, so we don't need to do so again.  You should instead inherit from HideC before AdvancedIncSubtensor to prevent C code leakage from the CPU version.