[PATCH] D122524: [clang][AVR] Emit proper warnings

Ben Shi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 28 10:23:14 PDT 2022


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



More information about the cfe-commits mailing list