[PATCH] D110455: DebugInfo: Use clang's preferred names for integer types

2021-10-02 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110455/new/ https://reviews.llvm.org/D110455 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cg

[clang] dd3caa9 - [clang-format] [docs] [NFC] improve clarity in the QualifierAlignment warning

2021-10-02 Thread via cfe-commits
Author: mydeveloperday Date: 2021-10-02T13:18:42+01:00 New Revision: dd3caa99bd874671a4c5c9657bd5e335dbfd3e0a URL: https://github.com/llvm/llvm-project/commit/dd3caa99bd874671a4c5c9657bd5e335dbfd3e0a DIFF: https://github.com/llvm/llvm-project/commit/dd3caa99bd874671a4c5c9657bd5e335dbfd3e0a.diff

[PATCH] D110801: [clang-format] [docs] [NFC] improve clarity in the QualifierAlignment warning

2021-10-02 Thread MyDeveloperDay via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGdd3caa99bd87: [clang-format] [docs] [NFC] improve clarity in the QualifierAlignment warning (authored by MyDeveloperDay). Changed prior to commit:

[clang] 3d209c7 - [clang-format] Constructor initializer lists format with pp directives

2021-10-02 Thread via cfe-commits
Author: Josh Learn Date: 2021-10-02T13:23:43+01:00 New Revision: 3d209c76ddb56db73def474450532813e39d95fa URL: https://github.com/llvm/llvm-project/commit/3d209c76ddb56db73def474450532813e39d95fa DIFF: https://github.com/llvm/llvm-project/commit/3d209c76ddb56db73def474450532813e39d95fa.diff LO

[PATCH] D109951: [clang-format] Constructor initializer lists format with pp directives

2021-10-02 Thread MyDeveloperDay via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3d209c76ddb5: [clang-format] Constructor initializer lists format with pp directives (authored by guitard0g, committed by MyDeveloperDay). Repositor

[PATCH] D110596: [CUDA] Move CUDA SDK include path further down the include search path.

2021-10-02 Thread Raul Tambre via Phabricator via cfe-commits
tambre added a comment. Thank you for your quick turnaround. I can confirm that this works for me on Debian Unstable with my vendored LLVM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110596/new/ https://reviews.llvm.org/D110596 ___

[PATCH] D77491: [Sema] Introduce BuiltinAttr, per-declaration builtin-ness

2021-10-02 Thread Raul Tambre via Phabricator via cfe-commits
tambre abandoned this revision. tambre added a comment. Abandoning since I don't think there's any further review/actions to be done here. D58531 would be the solution for the raised `pthread_create()` issue. Repository: rG LLVM Github Monorepo CHANGES SINC

[clang] ac21e39 - [clang-format] NFC 1% improvement in the overall clang-formatted status

2021-10-02 Thread via cfe-commits
Author: mydeveloperday Date: 2021-10-02T18:05:45+01:00 New Revision: ac21e3922b1220ef44c6faa7bbf832ac3fb9dda2 URL: https://github.com/llvm/llvm-project/commit/ac21e3922b1220ef44c6faa7bbf832ac3fb9dda2 DIFF: https://github.com/llvm/llvm-project/commit/ac21e3922b1220ef44c6faa7bbf832ac3fb9dda2.diff

[PATCH] D110833: [clang-format] Add ControlStatementsAndFunctionDefinitionsExceptControlMacros option to SpaceBeforeParens

2021-10-02 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added inline comments. Comment at: clang/docs/ClangFormatStyleOptions.rst:3649 ``SBPO_ControlStatementsExceptForEachMacros`` remains an alias for backward compatibility. crayroud wrote: > MyDeveloperDay wrote: > > Now I look back he

[PATCH] D77491: [Sema] Introduce BuiltinAttr, per-declaration builtin-ness

2021-10-02 Thread Yuanfang Chen via Phabricator via cfe-commits
ychen added a comment. In D77491#3038204 , @tambre wrote: > Abandoning since I don't think there's any further review/actions to be done > here. > D58531 would be the solution for the raised > `pthread_create()` issue.

[PATCH] D77491: [Sema] Introduce BuiltinAttr, per-declaration builtin-ness

2021-10-02 Thread Raul Tambre via Phabricator via cfe-commits
tambre added a comment. In D77491#3038277 , @ychen wrote: > In D77491#3038204 , @tambre wrote: > >> Abandoning since I don't think there's any further review/actions to be done >> here. >> D58531

[PATCH] D111000: [clang-format] allow clang-format to be passed a file of filenames so we can add a regression suite of "clean clang-formatted files" from LLVM

2021-10-02 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land. Basically okay, I would have made 3 commits out of it: 1. Add the function to clang-format 2. The code clean up of the python script 3. The additional file generation

[PATCH] D109727: [Driver] Remove unneeded *-suse-* triples

2021-10-02 Thread Aaron Puchert via Phabricator via cfe-commits
aaronpuchert added a comment. The problem here is that triples are not just used to look up the GCC installation, but the individual components have some meaning. Changing the OS vendor from "unknown" to "suse" doesn't worry me, but dropping "-gnu" might lead us to the problems described in D10

[PATCH] D110428: [AIX] Define WCHAR_T_TYPE as unsigned short on AIX for wchar.c test case.

2021-10-02 Thread Amy Kwan via Phabricator via cfe-commits
amyk updated this revision to Diff 376723. amyk added a comment. Addressed comment made by @daltenty. Update the patch to account set the wchar type to `unsigned short` only on 32-bit AIX. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110428/new/