benshi001 added a comment.

In D122524#3410542 <https://reviews.llvm.org/D122524#3410542>, @MaskRay wrote:

> I think it is excessive to add so many RUN lines. I do not understand much 
> about AVR -mcpu. That said, I created D122553 
> <https://reviews.llvm.org/D122553> for what I think should be done for the 
> `-c/-S/-fsyntax-only` condition. More tests would just be excessive.
>
> You may adjust the patch to do the rest cleanups/fixes.

I have made some changes based on your https://reviews.llvm.org/D122553, it 
seems you have reverted. I suggest you recommit, since I have fixed the 
failures of lacking avr-gcc.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122524/new/

https://reviews.llvm.org/D122524

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to