[PATCH] D50410: Removing -debug-info-macros from option suggestions test

Matt Davis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 16 08:33:42 PDT 2018


mattd added a comment.

In https://reviews.llvm.org/D50410#1201206, @acoomans wrote:

> @mattd @Sunil_Srivastava if you have access to the PS4 SDK, could you tell if the `-debug-info-macro` command line clang option is available? Thank you


Hi @acoomans,

The ps4 buildbot should be running with the same code that is available in the clang repo.  As far as I know, that bot uses the ps4 triple: x86_64-scei-ps4, and there is no special treatment for the -debug-info-macro flag under that triple in the codebase.  So, the flag should be available when testing, and you should be getting the "suggestion" in your tests.  You can try adding `x86_64-scei-ps4` triple to your RUN line, and see if things still pass.  As far as I can tell having the test for the -debug-macro-info suggestion is fine. I am curious, that failure was back in May, if you try a more recent build does the issue still show up?


Repository:
  rC Clang

https://reviews.llvm.org/D50410





More information about the cfe-commits mailing list