RFC: Removal of Nios2 backend

We (Intel) would like to propose removal of the Nios2 backend from the repository.

It is currently considered an experimental target. The implementation is not complete and months went by without anyone noticing it doesn’t build. See https://reviews.llvm.org/D56178

Due to internal priority changes, it doesn’t look like it will become more maintained in the near future. So I would like to propose removing it from the repository.

I will put together a patch for review shortly.

Patches up

Clang: https://reviews.llvm.org/D56690

LLVM: https://reviews.llvm.org/D56691

FWIW, I support this especially as it doesn’t build.

If someone wants to revive it, removing it won’t actually make that much harder (if at all) given that they’d need to clean up the build as well.

Are there any other (non-Intel) devs who contributed significantly or might have specific opinions about this?

Does it make more sense to this before or after the branch?

As far as I could tell, the only non-Intel contributions were from mechanical API updates or fixing include paths when files moved to other libraries for layering.

I’m happy to do it on whatever side of the branch people prefer.

+Hans Wennborg +tstellar@redhat.com for release thoughts…

FWIW, I support this especially as it doesn't build.

+1

If someone wants to revive it, removing it won't actually make that much harder (if at all) given that they'd need to clean up the build as well.

Are there any other (non-Intel) devs who contributed significantly or might have specific opinions about this?

Does it make more sense to this before or after the branch?

I think that we should remove non-building code before the branch.

-Hal

+1

-Chris

I don't really have a preference for doing it before or after the
release branch. If the patches are ready, landing them sounds good to
me.

Patches have been committed. r351230(clang) and r351231(llvm)

Hi Craig,

Do you want to write a note about this for the 8.0.x release notes?

Thanks,
Hans