[PATCH] D74104: Remove test dependency on the presence of an assembler

2020-03-09 Thread Douglas Yung via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG54d82255fd27: Change tests to use -S instead of -c to work when an external assembler is used… (authored by dyung). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D74104: Remove test dependency on the presence of an assembler

2020-03-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Interesting, indeed, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74104/new/ https://reviews.llvm.org/D74104 _

[PATCH] D74104: Remove test dependency on the presence of an assembler

2020-03-02 Thread Douglas Yung via Phabricator via cfe-commits
dyung added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74104/new/ https://reviews.llvm.org/D74104 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D74104: Remove test dependency on the presence of an assembler

2020-02-05 Thread Douglas Yung via Phabricator via cfe-commits
dyung created this revision. dyung added a reviewer: NoQ. dyung added a project: clang. Herald added a subscriber: Charusso. These tests currently call -c which requires an assembler, and will fail if your target defaults to an external assembler that is not present in the test environment. It d