[PATCH] D78163: [AVR][NFC] Move preprocessor tests to Preprocessor directory

Ayke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 07:05:25 PDT 2020


aykevl added a comment.

Honestly I don't know the convention but I agree with @rjmccall that the convention appears to be to not use a separate subdirectory (I don't see any other arch-specific subdirectories). So, should the files be moved to clang/test/Preprocessor/avr-*.c (e.g. clang/test/Preprocessor/avr-attiny104.c) while keeping `REQUIRES: avr-registered-target`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78163





More information about the llvm-commits mailing list