Details
Diff Detail
Event Timeline
Most patterns and several of the ones coming from lit.local.cfg are explained in TestingGuide.rst as well. On the other hand it makes sense to explain the lit specific substitutions in the lit docu, maybe just add an additional link to the TestingGuide.rst to reference this section for additional lit flags?
I'd probably go for a term like "pattern" instead of macro (there is no parameters or anything more complicated). The explanation for %t/%T is not helpful in the current form. Maybe something like 'temporary file name unique to the test'/'temporary directory unique to the test'.
Apart from that LGTM.
I had a hard time myself figuring out what tmpbase and tmpdir are. I have used your wording.
Somebody else needs to commit this as I don't have any rights for it.
When someone else commits for you, our practice is to attribute the correct author in the commit message.
This is more than a courtesy, because you retain copyright of any changes you have made. Therefore we prefer to use your actual name rather than a handle such as RedX2501, guibufolo, or Guilherme.
What name would you like us to use?
Thanks for the advice.
Please use Guilherme Bufolo as the name for the commit.
Am 04.04.2016 21:33 schrieb "Paul Robinson" <
Paul_Robinson@playstation.sony.com>:
probinson added a subscriber: probinson.
probinson added a comment.In http://reviews.llvm.org/D18752#390906, @RedX2501 wrote:
Somebody else needs to commit this as I don't have any rights for it.
When someone else commits for you, our practice is to attribute the
correct author in the commit message.
This is more than a courtesy, because you retain copyright of any changes
you have made. Therefore we prefer to use your actual name rather than a
handle such as RedX2501, guibufolo, or Guilherme.
What name would you like us to use?