benshi001 wrote: > For future reference, the avr16/avr32 devices do have SPM and SPMX features, > but the current xmega3 definition has not. > > Xmega3 is also used for modern attiny series which do not have SPM(X), so > that is correct. > > The avr64 devices use xmega2 which has SPM(X) defined, so that is also ok. > > I prefer to leave the avr16/avr32 devices unchanged (using xmega3 to be in > sync with gcc definitions), because the SPM instructions write flash and > users can use asm if they want that feature. In addition LLVM will never > generate these? > > For me this PR is complete now. @benshi001 add this comment to the commit > message?
It is OK to keep in sync with gcc, thanks. llvm-avr never generate unsupported features for devices. https://github.com/llvm/llvm-project/pull/144229 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits