[PATCH] D63685: [analyzer] exploded-graph-rewriter: Add support for range constraints.

NoQ created this revision. NoQ added reviewers: dcoughlin, xazax.hun, a_sidorin, rnkovacs, Szelethus, baloghadamsoftware, Charusso. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet. Herald added a project: clang. NoQ added a parent revision: D63684:

[PATCH] D63684: [analyzer] exploded-graph-rewriter: NFC: Extract some code into functions.

NoQ created this revision. NoQ added a reviewer: Charusso. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. Even though this code is not duplicated yet, follow-up patches will

[PATCH] D63438: [analyzer] print() JSONify: ProgramPoint revision

NoQ added a comment. Pls commit? ^.^ CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63438/new/ https://reviews.llvm.org/D63438 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D63462: [analyzer] JsonSupport: Escape escapes

NoQ added a comment. I guess let's add a test for the unicode problem that you're seeing. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63462/new/ https://reviews.llvm.org/D63462 ___ cfe-commits mailing list

[PATCH] D63649: AMDGPU: Fix target builtins for gfx10

arsenm closed this revision. arsenm added a comment. r364123 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63649/new/ https://reviews.llvm.org/D63649 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r364123 - AMDGPU: Fix target builtins for gfx10

Author: arsenm Date: Fri Jun 21 18:30:00 2019 New Revision: 364123 URL: http://llvm.org/viewvc/llvm-project?rev=364123=rev Log: AMDGPU: Fix target builtins for gfx10 This wasn't setting some of the features from older generations. Modified: cfe/trunk/lib/Basic/Targets/AMDGPU.cpp

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63626/new/ https://reviews.llvm.org/D63626

[clang-tools-extra] r364120 - [NFC] Marking test added in r363975 as unsupported on Windows.

Author: dyung Date: Fri Jun 21 18:09:00 2019 New Revision: 364120 URL: http://llvm.org/viewvc/llvm-project?rev=364120=rev Log: [NFC] Marking test added in r363975 as unsupported on Windows. This test references a path that does not exist on Windows causing it to emit different output from what

r364119 - [ODRHash] Skip some typedef types.

Author: rtrieu Date: Fri Jun 21 17:32:19 2019 New Revision: 364119 URL: http://llvm.org/viewvc/llvm-project?rev=364119=rev Log: [ODRHash] Skip some typedef types. In some cases, a typedef only strips aways a keyword for a type, keeping the same name as the root record type. This causes some

[PATCH] D63666: [clang-doc] Add templates to HTML generator

DiegoAstiazaran added inline comments. Comment at: clang-tools-extra/unittests/clang-doc/HTMLGeneratorTest.cpp:313-314 + + + + Brief description. juliehockett wrote: > Emitting extra div/p here? Add a check in the comment generation bit to > ensure you're not

[PATCH] D63666: [clang-doc] Add templates to HTML generator

DiegoAstiazaran updated this revision to Diff 206103. DiegoAstiazaran marked 4 inline comments as done. DiegoAstiazaran added a comment. Add FIXME comments for llvm::SmallString to std::string transition. Add check so ParagraphComment template is not generated if its content would be empty. Fix

[PATCH] D63681: [clang-scan-deps] Introduce the DependencyScanning library with the thread worker code and better error handling

arphaman created this revision. arphaman added reviewers: Bigcheese, aganea. Herald added subscribers: tschuett, dexonsmith, jkorous, mgorny. Herald added a project: clang. This patch extracts out the code that will powers the fast scanning worker into a new file in a new DependencyScanning

[PATCH] D63617: [COFF, ARM64] Fix encoding of __debugbreak

TomTan abandoned this revision. TomTan added a comment. The fix in LLVM was merged as https://reviews.llvm.org/rL364115. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63617/new/ https://reviews.llvm.org/D63617 ___

[PATCH] D63276: [AST] Add FunctionDecl::getParametersSourceRange()

nicolas added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63276/new/ https://reviews.llvm.org/D63276 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D63678: Fix test Clang :: Driver/cl-response-file.c for Solaris

rnk added a comment. In D63678#1554513 , @ro wrote: > Please have a look at what the autoconf manual has to say on echo vs. printf: Yikes, fair enough. I actually really wish lit had some support for the here-document pattern: cat

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

Szelethus marked an inline comment as done. Szelethus added inline comments. Comment at: clang/test/Analysis/track-control-dependency-conditions.cpp:1-8 +// RUN: %clang_analyze_cc1 %s -verify -DTRACKING_CONDITIONS \ +// RUN: -analyzer-config track-conditions=true \ +// RUN:

[PATCH] D63678: Fix test Clang :: Driver/cl-response-file.c for Solaris

ro added a comment. In D63678#1554508 , @rnk wrote: > I would almost prefer to XFAIL this on Solaris, I feel like the intent is > clearer with echo. Please have a look at what the autoconf manual has to say on echo vs. printf:

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

Szelethus added a comment. Woohoo! Thanks for everything, this is the most fun I've had working on this project! Let's wait for @xazax.hun to have the final say. In D62883#1554494 , @NoQ wrote: > It should be pretty easy to implement, just add your new

[PATCH] D63666: [clang-doc] Add templates to HTML generator

DiegoAstiazaran marked 2 inline comments as done. DiegoAstiazaran added inline comments. Comment at: clang-tools-extra/unittests/clang-doc/HTMLGeneratorTest.cpp:307-308 + + + +f juliehockett wrote: > Emitting an extra div here? The first div is emitted by the

Re: r363975 - [clang-tidy] Fail gracefully upon empty database fields

Hello Serge, This commit added broken test to the builder: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/26525 . . . Failing Tests (2): Clang Tools :: clang-tidy/empty-database.cpp . . . Please have a look? The builder was already red and did

[PATCH] D63678: Fix test Clang :: Driver/cl-response-file.c for Solaris

rnk added a comment. I would almost prefer to XFAIL this on Solaris, I feel like the intent is clearer with echo. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63678/new/ https://reviews.llvm.org/D63678 ___ cfe-commits mailing list

[PATCH] D63642: [analyzer] Add a debug analyzer config to place an event for each tracked condition

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Aha, yep, nice debug flag! Comment at: clang/test/Analysis/track-control-dependency-conditions.cpp:16 +// RUN: %clang_analyze_cc1 %s -verify \ +// RUN: -DTRACKING_CONDITIONS

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

NoQ added a comment. It should be pretty easy to implement, just add your new visitor to the list of default visitors in `findValidReport()`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62883/new/ https://reviews.llvm.org/D62883

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

NoQ accepted this revision. NoQ added a comment. Aha, ok, got it. I guess the official term is "error node" (where "error" means "warning"). Comment at: clang/test/Analysis/track-control-dependency-conditions.cpp:1-8 +// RUN: %clang_analyze_cc1 %s -verify

RE: r363985 - [test][Driver] Fix Clang :: Driver/cl-response-file.c

I see Reid has reverted your original fix so instead of directly submitting, I have put the patch up for review first at https://reviews.llvm.org/D63678. Douglas Yung -Original Message- From: Rainer Orth Sent: Friday, June 21, 2019 14:46 To: Yung, Douglas Cc:

[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

dyung added a comment. In D63600#1554029 , @rnk wrote: > This causes the test to fail on Windows: > http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/7712 > > There seems to be something wrong with the blamelist, so it didn't send > email.

[PATCH] D63678: Fix test Clang :: Driver/cl-response-file.c for Solaris

dyung created this revision. dyung added reviewers: ro, rnk, jkorous. dyung added a project: clang. Herald added subscribers: dexonsmith, fedor.sergeev. This is a follow up to D63600 that attempts to fix the test to work on all platforms including Windows (which

r364110 - Remove binary finally accidentially committed in r364109

Author: erichkeane Date: Fri Jun 21 15:31:59 2019 New Revision: 364110 URL: http://llvm.org/viewvc/llvm-project?rev=364110=rev Log: Remove binary finally accidentially committed in r364109 Removed: cfe/trunk/test/CodeGenCXX/target-builtin-error.o Removed:

Re: r364102 - Fix __has_cpp_attribute expansion to produce trailing L and (where

I added -P to remove the line markers which were introducing the colons. On Fri, Jun 21, 2019 at 2:58 PM Galina Kistanova via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Hello Richard, > > This commit added broken test to the builder: > >

r364109 - Ensure Target Features always_inline error happens in C++ cases.

Author: erichkeane Date: Fri Jun 21 15:29:32 2019 New Revision: 364109 URL: http://llvm.org/viewvc/llvm-project?rev=364109=rev Log: Ensure Target Features always_inline error happens in C++ cases. A handful of C++ cases as reported in PR42352 didn't actually give an error when always_inlining

r364108 - Fix has_attribute.cpp test on Windows after r364102

Author: rnk Date: Fri Jun 21 15:28:52 2019 New Revision: 364108 URL: http://llvm.org/viewvc/llvm-project?rev=364108=rev Log: Fix has_attribute.cpp test on Windows after r364102 Modified: cfe/trunk/test/Preprocessor/has_attribute.cpp Modified: cfe/trunk/test/Preprocessor/has_attribute.cpp

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

Szelethus added a comment. In D62883#1554339 , @NoQ wrote: > In D62883#1553248 , @Szelethus wrote: > > > - Add a `TODO:` in `trackExpressionValue` about maybe tracking conditions > > to all bug locations, rather

[PATCH] D63663: [clang-doc] Add html links to the parents of a RecordInfo

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:21 namespace { Anonymous namespace is used for functions when LLVM Coding Guidelines tells using static for same purpose. Comment at:

[PATCH] D63636: [PowerPC][Altivec] Fix offsets for vec_xl and vec_xst

jsji added a comment. A few questions. Comment at: lib/Headers/altivec.h:16364 signed short *__ptr) { - return *(unaligned_vec_sshort *)(__ptr + __offset); + signed char *Adjusted = (signed char *)__ptr + __offset; +

Re: r364102 - Fix __has_cpp_attribute expansion to produce trailing L and (where

Hello Richard, This commit added broken test to the builder: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/26522 . . . Failing Tests (2): Clang :: Preprocessor/has_attribute.cpp . . . Please have a look? The builder was already red and did not

[PATCH] D62883: [analyzer] Track terminator conditions on which a tracked expressions depends

NoQ added a comment. In D62883#1553248 , @Szelethus wrote: > - Add a `TODO:` in `trackExpressionValue` about maybe tracking conditions to > all bug locations, rather than only for tracked variables. What do you mean by "all bug locations"?

[PATCH] D63088: [clang-tidy] misc-unused-parameters: don't comment out parameter name for C code

This revision was automatically updated to reflect the committed changes. Closed by commit rL364106: [clang-tidy] misc-unused-parameters: dont comment out parameter name for C code (authored by mgehre, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed

[clang-tools-extra] r364106 - [clang-tidy] misc-unused-parameters: don't comment out parameter name for C code

Author: mgehre Date: Fri Jun 21 14:30:25 2019 New Revision: 364106 URL: http://llvm.org/viewvc/llvm-project?rev=364106=rev Log: [clang-tidy] misc-unused-parameters: don't comment out parameter name for C code Summary: The fixit `int square(int /*num*/)` yields `error: parameter name omitted`

[PATCH] D62825: [C++2a] Add __builtin_bit_cast, used to implement std::bit_cast

Quuxplusone added inline comments. Comment at: clang/include/clang/Basic/DiagnosticASTKinds.td:223 +def note_constexpr_bit_cast_invalid_type : Note< + "cannot constexpr evaluate a bit_cast with a " + "%select{union|pointer|member pointer|volatile|struct with a reference

[PATCH] D43576: Solution to fix PR27066 - Redefinition with same mangled name as another definition (dllexport and uuid)

rsmith added inline comments. Comment at: include/clang/AST/Decl.h:4303 + + StringLiteral *getSTLUuid() { return STLUuid; } +}; What does "STL" mean here? Comment at: include/clang/Sema/ParsedAttr.h:337-346 + /// Constructor for

[PATCH] D62825: [C++2a] Add __builtin_bit_cast, used to implement std::bit_cast

rsmith added inline comments. Comment at: clang/include/clang/Basic/DiagnosticASTKinds.td:223 +def note_constexpr_bit_cast_invalid_type : Note< + "cannot constexpr evaluate a bit_cast with a " + "%select{union|pointer|member pointer|volatile|struct with a reference member}0"

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 updated this revision to Diff 206075. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63082/new/ https://reviews.llvm.org/D63082 Files: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaChecking.cpp test/Analysis/misc-ps.c

[PATCH] D63508: make -frewrite-includes also rewrite conditions in #if/#elif

llunak added a comment. Implemented in the current version of the patch. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63508/new/ https://reviews.llvm.org/D63508 ___ cfe-commits mailing list

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5625 + "evaluate to 'true'">, + InGroup; + xbolva00 wrote: > aaron.ballman wrote: > > xbolva00 wrote: > > > aaron.ballman wrote: >

[PATCH] D63508: make -frewrite-includes also rewrite conditions in #if/#elif

llunak updated this revision to Diff 206070. llunak retitled this revision from "make -frewrite-includes handle __has_include wrapped in a macro" to "make -frewrite-includes also rewrite conditions in #if/#elif". llunak edited the summary of this revision. Repository: rC Clang CHANGES SINCE

[PATCH] D63666: [clang-doc] Add templates to HTML generator

juliehockett added inline comments. Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:59 + Items += + applyHTMLTemplate(HTMLTemplates::BlockItem, M.str().str().c_str()); +} Put `FIXME: Transition Members from llvm::SmallString to

[PATCH] D63256: [OpenCL] Split type and macro definitions into opencl-c-base.h

yaxunl added inline comments. Comment at: cfe/trunk/lib/Headers/opencl-c.h:13638 #if __OPENCL_C_VERSION__ >= CL_VERSION_2_0 -#ifndef ATOMIC_VAR_INIT -#define ATOMIC_VAR_INIT(x) (x) kzhuravl wrote: > Any reason this piece of code got completely removed? Removing

r364104 - clang-format a block; NFC

Author: gbiv Date: Fri Jun 21 13:49:47 2019 New Revision: 364104 URL: http://llvm.org/viewvc/llvm-project?rev=364104=rev Log: clang-format a block; NFC The indentation of the return here was off, and confusing as a result. Cleaned up a bit extra while I was in the area. Modified:

r364103 - PR42301: Abort cleanly if we encounter a huge source file rather than

Author: rsmith Date: Fri Jun 21 13:46:22 2019 New Revision: 364103 URL: http://llvm.org/viewvc/llvm-project?rev=364103=rev Log: PR42301: Abort cleanly if we encounter a huge source file rather than crashing. Ideally we wouldn't care about the size of a file so long as it fits in memory, but in

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

leonardchan updated this revision to Diff 206065. leonardchan marked 2 inline comments as done. leonardchan edited the summary of this revision. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63626/new/ https://reviews.llvm.org/D63626 Files:

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

leonardchan added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:665-668 +// We must also remove exception handling before attaching sample profile +// data. +MPM.addPass( +createModuleToPostOrderCGSCCPassAdaptor(PostOrderFunctionAttrsPass()));

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

chandlerc added a comment. FWIW, I think we can wait for a bug to be filed or report come in with some functional test case before we change any behavior here. I'm just suggesting how to make the test better below. Comment at: llvm/lib/Passes/PassBuilder.cpp:665-668 +//

r364102 - Fix __has_cpp_attribute expansion to produce trailing L and (where

Author: rsmith Date: Fri Jun 21 13:20:21 2019 New Revision: 364102 URL: http://llvm.org/viewvc/llvm-project?rev=364102=rev Log: Fix __has_cpp_attribute expansion to produce trailing L and (where necessary) leading whitespace. Simplify unit test and extend to cover no_unique_address attribute.

[PATCH] D63666: [clang-doc] Add templates to HTML generator

DiegoAstiazaran created this revision. DiegoAstiazaran added reviewers: juliehockett, jakehehrlich, lebedev.ri. DiegoAstiazaran added a project: clang-tools-extra. Herald added a subscriber: mgorny. Templates are added to create a stuctured HTML file. Commands in doxygen comments are still not

[PATCH] D63174: [clang][NewPM] Add RUNS for tests that produce slightly different IR under new PM

leonardchan added a comment. In D63174#1553096 , @craig.topper wrote: > I'm going to try to work on the X86 tests. Can we hold off on committing > those? Ok. For now I'll just commit the convergent.cl test separately. Repository: rG LLVM Github

[PATCH] D63161: Devirtualize destructor of final class.

This revision was automatically updated to reflect the committed changes. Closed by commit rL364100: Devirtualize destructor of final class. (authored by yamauchi, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5625 + "evaluate to 'true'">, + InGroup; + aaron.ballman wrote: > xbolva00 wrote: > > aaron.ballman wrote: > > > xbolva00 wrote: >

r364100 - Devirtualize destructor of final class.

Author: yamauchi Date: Fri Jun 21 13:04:29 2019 New Revision: 364100 URL: http://llvm.org/viewvc/llvm-project?rev=364100=rev Log: Devirtualize destructor of final class. Summary: Take advantage of the final keyword to devirtualize destructor calls. Fix

[PATCH] D63161: Devirtualize destructor of final class.

yamauchi updated this revision to Diff 206053. yamauchi marked 2 inline comments as done. yamauchi added a comment. Comments addressed. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63161/new/ https://reviews.llvm.org/D63161 Files: lib/CodeGen/CGExprCXX.cpp

[PATCH] D63256: [OpenCL] Split type and macro definitions into opencl-c-base.h

kzhuravl added subscribers: b-sumner, kzhuravl. kzhuravl added a comment. + @b-sumner Comment at: cfe/trunk/lib/Headers/opencl-c.h:13638-13640 -#ifndef ATOMIC_VAR_INIT -#define ATOMIC_VAR_INIT(x) (x) -#endif //ATOMIC_VAR_INIT Any reason this piece of code got

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

aaron.ballman added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5625 + "evaluate to 'true'">, + InGroup; + xbolva00 wrote: > aaron.ballman wrote: > > xbolva00 wrote: > > > aaron.ballman wrote: > > > > This one seems like it should

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5625 + "evaluate to 'true'">, + InGroup; + aaron.ballman wrote: > xbolva00 wrote: > > aaron.ballman wrote: > > > This one seems

[PATCH] D63663: [clang-doc] Add html links to the parents of a RecordInfo

juliehockett added a comment. General design thought: could we move the path name construction to `Serialize.cpp`? Essentially, move the logic that builds the path name from the namespaces and name to the serialize step (you'll have to add the `OutDirectory` to the `ClangDocContext` so that

[PATCH] D63082: [Diagnostics] Added support for -Wint-in-bool-context

aaron.ballman added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5609 +def warn_mul_in_bool_context : Warning< + "'*' in bool context, maybe you mean '&&'?">, + InGroup; xbolva00 wrote: > aaron.ballman wrote: > > I would appreciate

[PATCH] D63663: [clang-doc] Add html links to the parents of a RecordInfo

DiegoAstiazaran created this revision. DiegoAstiazaran added reviewers: juliehockett, jakehehrlich, lebedev.ri. DiegoAstiazaran added a project: clang-tools-extra. Herald added subscribers: kadircet, arphaman. tags are added for the parents of records. The link redirects to the parent's info

RE: r363985 - [test][Driver] Fix Clang :: Driver/cl-response-file.c

Hi Rainer, I figured it out. Because you are using printf, you no longer need the '\\c'. If I use the following RUN line, the test passes on both Windows/linux (I don't have solaris to test unfortunately): // RUN: printf '%%s\n' '/I%S\Inputs\cl-response-file\ /DFOO=2' > %t.rsp I can submit

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

leonardchan updated this revision to Diff 206041. leonardchan marked an inline comment as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63626/new/ https://reviews.llvm.org/D63626 Files: clang/test/CodeGen/pgo-sample.c Index:

[PATCH] D63626: [clang][NewPM] Remove exception handling before loading pgo sample profile data

leonardchan added a comment. In D63626#1553054 , @chandlerc wrote: > See inline comment, but I think we should just drop the testing of the > function attribute bit here rather than adjusting the pipeline. Ok. Removed the pipeline change for now. In

r364089 - Revert [test][Driver] Fix Clang :: Driver/cl-response-file.c

Author: rnk Date: Fri Jun 21 11:33:20 2019 New Revision: 364089 URL: http://llvm.org/viewvc/llvm-project?rev=364089=rev Log: Revert [test][Driver] Fix Clang :: Driver/cl-response-file.c This reverts r363985 (git commit d5f16d6cfccc4b0b13b6c01d16c673886d53e695) This test can't use printf on

[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

rnk added a comment. This causes the test to fail on Windows: http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/7712 There seems to be something wrong with the blamelist, so it didn't send email. I see this in the log from the previous build on that bot: Updating llvm to 363241

r364088 - [clang-scan-deps] print the dependencies to stdout

Author: arphaman Date: Fri Jun 21 11:24:55 2019 New Revision: 364088 URL: http://llvm.org/viewvc/llvm-project?rev=364088=rev Log: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB Differential Revision: https://reviews.llvm.org/D63579 Modified:

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

This revision was automatically updated to reflect the committed changes. Closed by commit rL364088: [clang-scan-deps] print the dependencies to stdout (authored by arphaman, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

RE: r363985 - [test][Driver] Fix Clang :: Driver/cl-response-file.c

Hi Rainer, I tried your patch and it still has problems. First, the "%s" was being replaced with the actual path by LIT, so I replaced that with "%%s" which then created the response file correctly, but now the response file contains this:

[clang-tools-extra] r364087 - Quote path to Python executable in case it has spaces

Author: rnk Date: Fri Jun 21 11:17:04 2019 New Revision: 364087 URL: http://llvm.org/viewvc/llvm-project?rev=364087=rev Log: Quote path to Python executable in case it has spaces These days Python 3 is typically installed into C:/Program Files, so cope with that. Similar to r364077 in

Re: [PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

Thanks, should hopefully be fixed by r364081. On Fri, 21 Jun 2019 at 01:12, Yvan Roux via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Hi Richard, > > This commit broke ARM bots, logs are available here: > > >

r364081 - Fix ARM buildbot.

Author: rsmith Date: Fri Jun 21 10:41:20 2019 New Revision: 364081 URL: http://llvm.org/viewvc/llvm-project?rev=364081=rev Log: Fix ARM buildbot. Modified: cfe/trunk/test/CodeGenCXX/tail-padding.cpp Modified: cfe/trunk/test/CodeGenCXX/tail-padding.cpp URL:

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

jfb added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; aaron.ballman wrote: > xbolva00 wrote: > > xbolva00 wrote: > > > jfb wrote: >

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

aaron.ballman added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; xbolva00 wrote: > xbolva00 wrote: > > jfb wrote: > > > xbolva00

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; xbolva00 wrote: > jfb wrote:

r364080 - [OPENMP]Fix PR42068: Vla type is not captured.

Author: abataev Date: Fri Jun 21 10:28:41 2019 New Revision: 364080 URL: http://llvm.org/viewvc/llvm-project?rev=364080=rev Log: [OPENMP]Fix PR42068: Vla type is not captured. If the variably modified type is declared outside of the captured region and then used in the cast expression along with

r364078 - Ensure that top-level QualType objects also have a "kind" field when dumping the AST to JSON.

Author: aaronballman Date: Fri Jun 21 10:14:25 2019 New Revision: 364078 URL: http://llvm.org/viewvc/llvm-project?rev=364078=rev Log: Ensure that top-level QualType objects also have a "kind" field when dumping the AST to JSON. Modified: cfe/trunk/lib/AST/JSONNodeDumper.cpp

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; jfb wrote: > xbolva00 wrote:

[PATCH] D63180: [clang-doc] Adds HTML generator

juliehockett added inline comments. Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:87 +std::string CloseTag = I.SelfClosing ? "/>" : ">"; +writeLine("<" + I.Name + Attrs.str() + CloseTag, OS); + } else if (I.Kind == "HTMLEndTagComment") { You

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

jfb added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; xbolva00 wrote: > Quuxplusone wrote: > > xbolva00 wrote: > > > jfb wrote: > >

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; Quuxplusone wrote: > xbolva00

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

Quuxplusone added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10929 +// Do not diagnose if xor keyword is used. +if (ExprStr.find("xor") != llvm::StringRef::npos) + return; xbolva00 wrote: > jfb wrote: > > Doesn't this match any expression

[PATCH] D62376: [ASTImporter] Mark erroneous nodes in shared st

martong added a comment. Alexei, Thanks for the review! I have provided test cases for the 3 previous patches on which this one depends on. I will provide additional tests next week for this one, and of course will address the other comments. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10983 + +S.Diag(Loc, diag::note_xor_used_as_pow_silence) << ("0x2 ^ " + RHSStr); + } else if (LeftSideValue == 10) { Quuxplusone wrote: > Do you

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

xbolva00 marked 3 inline comments as done. xbolva00 added inline comments. Comment at: include/clang/Basic/DiagnosticGroups.td:508 def Varargs : DiagGroup<"varargs">; +def XorUsedAsPow : DiagGroup<"xor-used-as-pow">; jfb wrote: > Does this match the naming

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

Quuxplusone added inline comments. Comment at: lib/Sema/SemaExpr.cpp:10950 + + // Do not diagnose binary literals. + if (ExprStr.find("0b") != llvm::StringRef::npos) I would rather see this whole section as a three-liner: // Do not diagnose binary, octal,

[PATCH] D62375: [ASTImporter] Mark erroneous nodes in from ctx

martong added a comment. Alexei, Thank you for the review, I have added a test which demonstrates the changes. By tracking the import paths and cycles we implement a very strict error handling mechanism, but this seems to be the way to avoid reaching any faulty AST nodes for the ASTImporter

[PATCH] D62375: [ASTImporter] Mark erroneous nodes in from ctx

martong updated this revision to Diff 206022. martong added a comment. - Add test ErrorPropagatesThroughImportCycles - Change existing test to new behaviour Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62375/new/ https://reviews.llvm.org/D62375

[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

jfb added inline comments. Comment at: include/clang/Basic/DiagnosticGroups.td:508 def Varargs : DiagGroup<"varargs">; +def XorUsedAsPow : DiagGroup<"xor-used-as-pow">; Does this match the naming that GCC ended up with? Comment at:

[PATCH] D62373: [ASTImporter] Store import errors for Decls

martong updated this revision to Diff 206020. martong added a comment. - Remove unused include and using Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62373/new/ https://reviews.llvm.org/D62373 Files: clang/include/clang/AST/ASTImporter.h

[PATCH] D63649: AMDGPU: Fix target builtins for gfx10

rampitec accepted this revision. rampitec added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63649/new/ https://reviews.llvm.org/D63649 ___ cfe-commits mailing list

[PATCH] D62335: [OpenCL][PR41963] Add overloads of old atomics with generic pointer type in C++ mode

This revision was automatically updated to reflect the committed changes. Closed by commit rL364071: [OpenCL][PR41963] Add generic addr space to old atomics in C++ mode (authored by stulova, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[PATCH] D63603: [ASTImporter] Propagate error from ImportDeclContext

martong updated this revision to Diff 206018. martong added a comment. - Remove formatv b/c it can't handle braces in code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63603/new/ https://reviews.llvm.org/D63603 Files:

[PATCH] D62373: [ASTImporter] Store import errors for Decls

martong updated this revision to Diff 206017. martong added a comment. - Remove formatv b/c it can't handle braces in code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62373/new/ https://reviews.llvm.org/D62373 Files:

r364071 - [OpenCL][PR41963] Add generic addr space to old atomics in C++ mode

Author: stulova Date: Fri Jun 21 09:19:16 2019 New Revision: 364071 URL: http://llvm.org/viewvc/llvm-project?rev=364071=rev Log: [OpenCL][PR41963] Add generic addr space to old atomics in C++ mode Add overloads with generic address space pointer to old atomics. This is currently only added for

r364067 - Print more type node information when dumping the AST to JSON.

Author: aaronballman Date: Fri Jun 21 09:06:09 2019 New Revision: 364067 URL: http://llvm.org/viewvc/llvm-project?rev=364067=rev Log: Print more type node information when dumping the AST to JSON. Added: cfe/trunk/test/AST/ast-dump-types-json.cpp Modified:

  1   2   >