tomtor wrote:

> > @benshi001 @Patryk27 Could you review this?
> 
> As I have told you several days ago, each functional change requires unit 
> tests, your PR involves
> 
>     1. clang driver
> 
>     2. clang builtin macro
> 
>     3. avr backend codegen
> 
> 
> Each part needs unit tests to show what are affected.

@benshi001 Can you point me to existing tests which vary the sub architecture 
of AVR? I would be happy to add to these tests the new architectures.

https://github.com/llvm/llvm-project/pull/146244
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to