[PATCH] D45357: [XRay][llvm+clang] Consolidate attribute list files

2018-04-08 Thread Dean Michael Berris via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL329543: [XRay][llvm+clang] Consolidate attribute list files (authored by dberris, committed by ). Changed prior to commit: https://reviews.llvm.org/D45357?vs=141578=141581#toc Repository: rL LLVM

[PATCH] D45357: [XRay][llvm+clang] Consolidate attribute list files

2018-04-08 Thread Dean Michael Berris via Phabricator via cfe-commits
dberris updated this revision to Diff 141578. dberris added a comment. - fixup: fix typo https://reviews.llvm.org/D45357 Files: clang/include/clang/Basic/LangOptions.h clang/include/clang/Basic/XRayLists.h clang/include/clang/Driver/Options.td clang/include/clang/Driver/XRayArgs.h

[PATCH] D45357: [XRay][llvm+clang] Consolidate attribute list files

2018-04-06 Thread Vlad Tsyrklevich via Phabricator via cfe-commits
vlad.tsyrklevich accepted this revision. vlad.tsyrklevich added inline comments. This revision is now accepted and ready to land. Comment at: llvm/docs/XRayExample.rst:196 +``-fxray-attr-list=`` flag to clang. You can have multiple files, each defining +different sets of

[PATCH] D45357: [XRay][llvm+clang] Consolidate attribute list files

2018-04-06 Thread Dean Michael Berris via Phabricator via cfe-commits
dberris created this revision. dberris added reviewers: echristo, vlad.tsyrklevich, eugenis. This change consolidates the always/never lists that may be provided to clang to externally control which functions should be XRay instrumented by imbuing attributes. The files follow the same format as