MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

Thanks! The convention is to add `[test] ` to test-only patches.

You can change the Phabricator title, then run `arc amend` to amend the local 
commit.
I believe on Debian testing `arc` is somewhat broken with php 8, so you may 
need to get `arc` from https://github.com/phacility/arcanist (`bin/arc`)



================
Comment at: clang/test/Preprocessor/file_test.c:13
 
 filename: __FILE__
+#include "./Inputs/./include-file-test/file_test.h"
----------------
This `./` line needs a comment about what is being tested.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127009

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

Reply via email to