This is an archive of the discontinued LLVM Phabricator instance.

Change 'clang-test' to 'check-clang' on the hacking webpage
ClosedPublic

Authored by acoomans on Jul 19 2018, 9:02 AM.

Details

Summary

The command to run tests was previously changed from 'clang-test' to 'check-clang'; with 'clang-test' remaining available as a legacy alias (see git-svn-id: https://reviews.llvm.org/rC159483 91177308-0d34-0410-b5e6-96231b3b80d8).

This commit changes the www documentation accordingly.

Diff Detail

Event Timeline

acoomans created this revision.Jul 19 2018, 9:02 AM
acoomans edited the summary of this revision. (Show Details)Jul 19 2018, 9:12 AM
JDevlieghere accepted this revision.Aug 9 2018, 8:16 AM
JDevlieghere added a subscriber: JDevlieghere.

Looks good, thanks!

PS: I don't think it hurts, but it's sufficient to add cfe-commits as a subscriber. I believe it's an automation account (so it won't be able to accept on phab) and replies via e-mail are posted under the corresponding user's account.

This revision is now accepted and ready to land.Aug 9 2018, 8:16 AM

@JDevlieghere Thanks! subscribers -> makes sense, will do

Hey Arnaud, let me know if you want me to commit this for you.

DavidSpickett closed this revision.Sep 21 2023, 1:47 AM
DavidSpickett added a subscriber: DavidSpickett.

I updated this to the monorepo layout and landed it: https://github.com/llvm/llvm-project/commit/604a231881a0e8f204a6d3337bfa3e797e911e34

Given the long time this has been here and the last activity of Arnaud here I've credited them by name in the commit message as was done for their other changes.

Herald added a project: Restricted Project. · View Herald TranscriptSep 21 2023, 1:47 AM