[PATCH] D56990: Bugfix for Replacement of tied operand of inline asm

xiangzhangllvm added a comment. Hi dears, Could you please help me merge the patch. Thank you! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56990/new/ https://reviews.llvm.org/D56990 ___ cfe-commits mailing list cf

r353802 - [X86] Use the new unaligned vector typedefs for the loadu/storeu intrinsics pointer arguments.

Author: ctopper Date: Mon Feb 11 23:44:40 2019 New Revision: 353802 URL: http://llvm.org/viewvc/llvm-project?rev=353802&view=rev Log: [X86] Use the new unaligned vector typedefs for the loadu/storeu intrinsics pointer arguments. This matches what gcc does and what was suggested by rnk in PR20670

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

Same thing for me with our downstream build bots. When we compile clang with gcc 7.4 and then run the testcase I get this output define dso_local i64 @test__shiftleft128(i64 %l, i64 %h, i8 %d) local_unnamed_addr #0 { entry: %0 = zext i64 %l to i128 %1 = zext i64 %h to i128 %2 = shl nuw

[PATCH] D57736: [WebAssembly] Bulk memory intrinsics and builtins

tlively added inline comments. Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:29 +// Bulk memory builtins +TARGET_BUILTIN(__builtin_wasm_memory_init, "vIiv*ii", "", "bulk-memory") +TARGET_BUILTIN(__builtin_wasm_data_drop, "vIi", "", "bulk-memory")

[PATCH] D57736: [WebAssembly] Bulk memory intrinsics and builtins

tlively updated this revision to Diff 186393. tlively marked 12 inline comments as done. tlively added a comment. - Address comments - Use `Ui` in builtin signatures - Remove unnecessary intrinsic polymorphism - Tweak intrinsic properties Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D58095: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

stephanemoore marked an inline comment as done. stephanemoore added inline comments. Comment at: clang-tools-extra/test/clang-tidy/google-objc-function-naming.m:10 +// function would be declared in a system header. +int printf(const char *, ...); // NOLINT(google-objc-function-n

[PATCH] D58095: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

stephanemoore created this revision. Herald added subscribers: cfe-commits, xazax.hun. Herald added a project: clang. Implicit functions are outside the control of source authors and should be exempt from style restrictions. Tested via running clang tools tests. This is an amended followup to ht

[PATCH] D58089: Add missing library dependencies in CMakeLists.txt

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58089/new/ https://reviews.llvm.org/D58089 ___

[PATCH] D58094: Fix -Wnonportable-include-path suppression for header maps with absolute paths.

vsapsai marked an inline comment as done. vsapsai added inline comments. Comment at: clang/lib/Lex/HeaderSearch.cpp:388 } else { Result = HS.getFileMgr().getFile(Dest); } I have considered changing this to `.getFile(Dest, /*OpenFile=*/true)` so that t

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

rsmith added a comment. In D57984#1394067 , @ABataev wrote: > In D57984#1394050 , @rsmith wrote: > > > @ABataev Is it intentional that we do not propagate `Allowed` through > > labels? For example: > > > > void f

[PATCH] D58094: Fix -Wnonportable-include-path suppression for header maps with absolute paths.

vsapsai created this revision. vsapsai added reviewers: dexonsmith, bruno. Herald added a subscriber: jkorous. In `DirectoryLookup::LookupFile` parameter `HasBeenMapped` doesn't cover the case when clang finds a file through a header map but doesn't remap the lookup filename because the target pat

[PATCH] D10833: Retrieve BinaryOperator::getOpcode and BinaryOperator::getOpcodeStr via libclang and its python interface

dyhe83 added a comment. In D10833#970906 , @milianw wrote: > still looks good to me. can someone else please review and commit this? ping Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D10833/new/ https://reviews.llvm.org/

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

JDevlieghere added a comment. In D58072#1393817 , @bruno wrote: > Not really. Would making only the `attachTo*` methods virtual enough though? You mean making them **not** virtual? They're the only ones I don't override. Repository: rC Clang CHANGES

[PATCH] D57497: [RISCV] Passing small data limitation value to RISCV backend

efriedma added a comment. > Did you mean declare as a target feature in RISCV.td or I misunderstanding > something? That's sort of the right idea, but I don't think it works in this context because we aren't trying to change the generated code for a function; we actually need to stick the glob

[PATCH] D57497: [RISCV] Passing small data limitation value to RISCV backend

shiva0217 added a comment. In D57497#1388350 , @apazos wrote: > If this is a target flag in GCC, shouldn't we make it a LLVM Target feature > and pass it as -mattr, just like done for mrelax? Hi Ana, It seems that most of the -mattr features only obtain

r353791 - Fix VS2015 build problem introduced by r353370.

Author: dyung Date: Mon Feb 11 18:17:51 2019 New Revision: 353791 URL: http://llvm.org/viewvc/llvm-project?rev=353791&view=rev Log: Fix VS2015 build problem introduced by r353370. Modified: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/SMTConstraintManager.h Modified: cfe/trunk

[PATCH] D58091: Customize warnings for missing built-in type

jdoerfert created this revision. jdoerfert added reviewers: jyknight, lebedev.ri, aaron.ballman, bcain. Herald added subscribers: jfb, bollu, krytarowski, emaste. Herald added a project: clang. If we detect a built-in declaration for which we cannot derive a type matching the pattern in the Builti

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

ABataev added a comment. In D57984#1394050 , @rsmith wrote: > @ABataev Is it intentional that we do not propagate `Allowed` through labels? > For example: > > void f() { > #pragma omp barrier // ok > > label: > #pragma omp barrier // error,

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

rsmith added a comment. @ABataev Is it intentional that we do not propagate `Allowed` through labels? For example: void f() { #pragma omp barrier // ok label: #pragma omp barrier // error, "cannot be an immediate substatement" label: ; #pragma omp barrier // ok } ?

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

Thank you for the .ll files! the -4.ll file you sent me contains: define dso_local i64 @"?test__shiftleft128@@YA_K_K0E@Z"(i64 %l, i64 %h, i8 %d) local_unnamed_addr #0 { entry: %0 = zext i64 %h to i128 %1 = shl nuw i128 %0, 64 %2 = zext i64 %l to i128 %3 = or i128 %1, %2 %4 = and i8 %d,

[PATCH] D58089: Add missing library dependencies in CMakeLists.txt

comex created this revision. comex added reviewers: jkorous, phosek. Herald added subscribers: cfe-commits, kadircet, arphaman, ioeric, ilya-biryukov, mgorny. Herald added a project: clang. Fixes build in BUILD_SHARED_LIBS mode. Removes the "DEPENDS clangdXpcJsonConversions" line as LINK_LIBS al

[PATCH] D57930: [Driver] Verify GCCInstallation is valid

danielmentz marked an inline comment as done. danielmentz added inline comments. Comment at: clang/lib/Driver/ToolChains/Linux.cpp:259-260 if (GCCInstallation.getParentLibPath().find("opt/rh/devtoolset") != StringRef::npos) // With devtoolset on RHEL, we want to

[PATCH] D54978: Move the SMT API to LLVM

brzycki added a comment. In D54978#1393968 , @thakis wrote: > Do you understand why the default matters for you? You seem to explicitly > disable the setting, and you still get Z3 as part of your build. Did you make > a clean build dir before turning it

[PATCH] D57976: -gmodules: Don't emit incomplete breadcrumbs pointing to nonexistant PCM files.

dblaikie added a comment. In D57976#1393976 , @aprantl wrote: > In D57976#1393939 , @dblaikie wrote: > > > This, I guess, is part of the impact of moving towards explicit modules > > (-fmodule-name is for building

Re: r353370 - Generalised the SMT state constraints

Hi Douglas, thank you for the report. We are discussing the issue in https://reviews.llvm.org/D54975. @michaelplatings said he found a solution and I asked him to run the tests locally. If it indeed fixes the issue, I'd say we push it. Em seg, 11 de fev de 2019 às 20:13, escreveu: > Hi Mikha

[PATCH] D57976: -gmodules: Don't emit incomplete breadcrumbs pointing to nonexistant PCM files.

aprantl added a comment. In D57976#1393939 , @dblaikie wrote: > This, I guess, is part of the impact of moving towards explicit modules > (-fmodule-name is for building a module with that name, right?)? That option is overloaded. It's used to specify th

RE: r353370 - Generalised the SMT state constraints

Hi Mikhail, Your commit seems to be causing a build failure on our internal Windows build bot that uses Visual Studio 2015. Can you take a look? C:\src\upstream\llvm_clean\tools\clang\include\clang/StaticAnalyzer/Core/PathSensitive/SMTConstraintManager.h(23): error C2872: 'ConstraintSMTTy': amb

[PATCH] D54978: Move the SMT API to LLVM

thakis added a comment. > In D54978#1393552 , @ddcc wrote: > >> The likely reason for this versioning problem is that the current versioning >> implementation in FindZ3.cmake is best-effort only. > > > Thank you @ddcc for this explanation. If that's the

[PATCH] D57976: -gmodules: Don't emit incomplete breadcrumbs pointing to nonexistant PCM files.

dblaikie added a comment. This, I guess, is part of the impact of moving towards explicit modules (-fmodule-name is for building a module with that name, right?)? With explicit modules there is the option for modular code generation - which doesn't require any specific DWARF consumer support (s

[PATCH] D56571: [RFC prototype] Implementation of asm-goto support in clang

jyu2 updated this revision to Diff 186356. jyu2 added a comment. I find some ambiguous error for asm-goto out of scope. Instead call VerifyIndirectOrAsmJumps one time, call that function twice one for indirect-goto one for asm-goto. Let me know if you see any problems. Thanks for all the review

[PATCH] D57236: [ASTImporter] Unify redecl chain tests as type parameterized tests

a_sidorin added a comment. Hi Gabor, The refactoring looks great. I have some minor comments inline. Comment at: unittests/AST/ASTImporterTest.cpp:3549 + void TypedTest_ImportDefinitionThenPrototype() { +Decl *FromTU0 = getTuDecl(getDefinition(), Lang_CXX, "input0.cc"); +

r353765 - Add a new attribute, fortify_stdlib

Author: epilk Date: Mon Feb 11 15:21:39 2019 New Revision: 353765 URL: http://llvm.org/viewvc/llvm-project?rev=353765&view=rev Log: Add a new attribute, fortify_stdlib This attribute applies to declarations of C stdlib functions (sprintf, memcpy...) that have known fortified variants (__sprintf_c

[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

erik.pilkington marked 8 inline comments as done. erik.pilkington added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:987-989 +The first argument to the attribute is the type passed to +`__builtin_object_size`, and the second is the flag that the fortified fo

[PATCH] D57855: [analyzer] Reimplement checker options

Szelethus marked 5 inline comments as done. Szelethus added inline comments. Comment at: include/clang/StaticAnalyzer/Checkers/Checkers.td:384 + CheckerOptions<[ +CmdLineOption It would be great if there were a way to define options once and reuse them. I think it's possible

[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

This revision was automatically updated to reflect the committed changes. Closed by commit rL353765: Add a new attribute, fortify_stdlib (authored by epilk, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D

[PATCH] D57855: [analyzer] Reimplement checker options

Szelethus updated this revision to Diff 186350. Szelethus added a comment. - Removed the accidentally added `nullability:Optimistic` entry, - Reworded `unix.DynamicMemoryModeling:Optimistic`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57855/new/ https://reviews.llvm.org/D57855 Files

[PATCH] D58069: [Sema] Mark GNU compound literal array init as an rvalue.

NoQ added a comment. > The analyzer test change maybe indicates we could simplify the analyzer code > a little with this fix? Apparently a hack was added to support lvalues in > initializers in r315750, but I'm not really familiar with the relevant code. Nope, unfortunately, not much can be sim

[PATCH] D54881: [clang-format] Prevent Clang-Format from editing leading whitespace on lines outside of the format range

russellmcc added a comment. Bump! Thanks for the consideration! Again, this is preventing us from rolling out automated style checking on diffs at work. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54881/new/ https://reviews.llvm.org/D54881

[PATCH] D57850: [analyzer] Emit an error rather than assert on invalid checker option input

Szelethus updated this revision to Diff 186346. Szelethus added a comment. - Added a convenience method to `CheckerManager` for reporting invalid user input - Edited the error message to be more flexible CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57850/new/ https://reviews.llvm.org/

[PATCH] D40988: Clang-format: add finer-grained options for putting all arguments on one line

russellmcc added a comment. Bump! Still watching this space - thanks so much for your time! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D40988/new/ https://reviews.llvm.org/D40988 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

r353762 - [Sema] Mark GNU compound literal array init as an rvalue.

Author: efriedma Date: Mon Feb 11 14:54:27 2019 New Revision: 353762 URL: http://llvm.org/viewvc/llvm-project?rev=353762&view=rev Log: [Sema] Mark GNU compound literal array init as an rvalue. Basically the same issue as string init, except it didn't really have any visible consequences before I

[PATCH] D58069: [Sema] Mark GNU compound literal array init as an rvalue.

This revision was automatically updated to reflect the committed changes. efriedma marked an inline comment as done. Closed by commit rC353762: [Sema] Mark GNU compound literal array init as an rvalue. (authored by efriedma, committed by ). Changed prior to commit: https://reviews.llvm.org/D580

[PATCH] D58069: [Sema] Mark GNU compound literal array init as an rvalue.

efriedma marked 2 inline comments as done. efriedma added inline comments. Comment at: lib/Sema/SemaInit.cpp:172-173 + E = PE->getSubExpr(); +else if (UnaryOperator *UO = dyn_cast(E)) + E = UO->getSubExpr(); +else if (GenericSelectionExpr *GSE = dyn_cast(E)) ---

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

This revision was automatically updated to reflect the committed changes. Closed by commit rL353761: [WebAssembly] Make thread-related options consistent (authored by aheejin, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SIN

r353761 - [WebAssembly] Make thread-related options consistent

Author: aheejin Date: Mon Feb 11 14:47:50 2019 New Revision: 353761 URL: http://llvm.org/viewvc/llvm-project?rev=353761&view=rev Log: [WebAssembly] Make thread-related options consistent Summary: There have been three options related to threads and users had to set all three of them separately to

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

bruno added a comment. Not really. Would making only the `attachTo*` methods virtual enough though? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58072/new/ https://reviews.llvm.org/D58072 ___ cfe-commits mailing lis

[PATCH] D57265: [PM/CC1] Add -f[no-]split-cold-code CC1 options to toggle splitting

fedor.sergeev added a comment. In D57265#1393453 , @vsk wrote: > > I'd prefer not adding this kind of state to PassBuilder. SplitColdCode is > > soemthing that refers to the construction of one particular pipeline, not > > to pipeline-building in general

[clang-tools-extra] r353760 - [NFC][clangd] Remove unused lambda capture

Author: thegameg Date: Mon Feb 11 14:36:47 2019 New Revision: 353760 URL: http://llvm.org/viewvc/llvm-project?rev=353760&view=rev Log: [NFC][clangd] Remove unused lambda capture Avoid this warning: llvm/clang-tools-extra/clangd/ClangdServer.cpp:365:23: warning: lambda capture 'this' is not used

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh updated this revision to Diff 186338. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58074/new/ https://reviews.llvm.org/D58074 Files: include/clang/AST/OpenMPClause.h include/clang/Basic/DiagnosticParseKinds.td include/clang/Basic/DiagnosticSemaKinds.td include/clang/Basic

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

rsmith updated this revision to Diff 186337. rsmith marked an inline comment as done. rsmith added a comment. Address a couple of review comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57984/new/ https://reviews.llvm.org/D57984 Files: include/clang/AST

[PATCH] D57850: [analyzer] Emit an error rather than assert on invalid checker option input

Szelethus marked 2 inline comments as done. Szelethus added inline comments. Comment at: include/clang/Basic/DiagnosticDriverKinds.td:304-305 def err_analyzer_config_unknown : Error<"unknown analyzer-config '%0'">; +def err_analyzer_checker_option_invalid_input : Error< + "inva

[PATCH] D58069: [Sema] Mark GNU compound literal array init as an rvalue.

rsmith accepted this revision. rsmith added inline comments. This revision is now accepted and ready to land. Comment at: lib/Sema/SemaInit.cpp:172-173 + E = PE->getSubExpr(); +else if (UnaryOperator *UO = dyn_cast(E)) + E = UO->getSubExpr(); +else if (GenericSe

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh created this revision. lildmh added reviewers: ABataev, hfinkel, Meinersbur. lildmh added a project: OpenMP. Herald added subscribers: cfe-commits, guansong. Herald added a project: clang. This patch implements the parsing and sema support for OpenMP map clauses with potential user-defined

[PATCH] D57890: [analyzer] Fix in self assignment checker

NoQ added a comment. Sounds reasonable, but it also sounds like something that should be reproducible on the upstream clang. Do you have a code snippet that causes the problematic AST to appear? Even if we don't have the false positive up here in upstream, Is it something we can test via `-anal

[PATCH] D58067: [Analyzer] Crash fix for FindLastStoreBRVisitor

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Aha, right, indeed, thanks, nice! We should eventually remove `operator==()` from the `SVal` class because it doesn't do anything you'd possibly imagine it to do. However we do need to come up with

[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM aside from some very minor nits. Comment at: clang/include/clang/Basic/AttrDocs.td:987-989 +The first argument to the attribute is the type passed to +`__builtin_object_size`, and the second is the flag t

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

JDevlieghere added a comment. In D58072#1393640 , @bruno wrote: > How much of the `ModuleDependencyCollector` will be reused as is by LLDB? I > wonder about the tradeoff versus inheriting from `DependencyCollector` > directly. We reuse the `attachTo` m

[PATCH] D57896: Variable names rule

rupprecht added a comment. In D57896#1391925 , @hubert.reinterpretcast wrote: > In D57896#1391611 , @zturner wrote: > > > Is this actually any better? Whereas before we can’t differentiate type > > names and vari

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

bruno added a comment. How much of the `ModuleDependencyCollector` will be reused as is by LLDB? I wonder about the tradeoff versus inheriting from `DependencyCollector` directly. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58072/new/ https://reviews.llvm.org/

[PATCH] D17444: PR26672: [MSVC] Clang does not recognize "static_assert" keyword in C mode

aaron.ballman added a comment. > Also, I suppose this is really actually an -fms-compatibility thing not > -fms-extensions because it takes an identifier from outside the implementer's > namespace. I just verified, it really does take the identifier and it does not treat `static_assert` as a m

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

JDevlieghere created this revision. JDevlieghere added reviewers: bruno, vsapsai. Herald added a project: clang. For reproducers in LLDB we want to hook up into the existing clang infrastructure. To make that happen we need to be able to override the ModuleDependencyCollector's methods. Reposi

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

Hello Nico, This builders fail on your test as well - http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/15736 , http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/4242. Please find attached the 2 temp files you can use to reliably run against your FileCh

[PATCH] D54978: Move the SMT API to LLVM

brzycki added a comment. In D54978#1392464 , @Szelethus wrote: > Shouldn't that be off by default? The default for `LLVM_ENABLE_Z3_SOLVER` depends entirely on what CMake detects from `find_package()`. Here is the relevant code in `llvm/CMakeLists.txt`:

[PATCH] D57655: clang-format with UseTab: Always sometimes doesn't insert the right amount of tabs.

hyklv added a comment. In D57655#1382437 , @lebedev.ri wrote: > Test? > Is this a fix, or a new formatting style? a fix. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57655/new/ https://reviews.llvm.org/D57655 _

[PATCH] D57655: clang-format with UseTab: Always sometimes doesn't insert the right amount of tabs.

hyklv added a comment. In D57655#1382855 , @MyDeveloperDay wrote: > The reviewers set a high barrier of entry, you need to at least start by > adding some unit tests (or someone will break your code in the future), if > you think it was a bug it might b

[PATCH] D57655: clang-format with UseTab: Always sometimes doesn't insert the right amount of tabs.

hyklv updated this revision to Diff 186315. hyklv marked an inline comment as done. hyklv added a comment. I removed accidental changes in the copyright notice, updated the code to insert one space instead of one tab. Added a test case for the alignment. Repository: rC Clang CHANGES SINCE LA

[PATCH] D17444: PR26672: [MSVC] Clang does not recognize "static_assert" keyword in C mode

rnk added a comment. In D17444#1393509 , @thakis wrote: > Given that we already have interposition headers and users already have to > hold the compiler right, does it matter if we have one more interposing > header? I'd say it's one more bit of techni

[PATCH] D49863: [istream] Fix error flags and exceptions propagated from input stream operations

ldionne added a comment. Herald added a subscriber: jkorous. I would like to merge this even though http://wg21.link/p1264 has not been voted into C++ yet, because this is arguably a bug and libc++ differs from other stdlibs in this regard. Once http://wg21.link/p1264 is merged into C++ in one

[PATCH] D54978: Move the SMT API to LLVM

ddcc added subscribers: delcypher, ddcc. ddcc added a comment. The likely reason for this versioning problem is that the current versioning implementation in FindZ3.cmake is best-effort only: among other conditions, if the z3 binary is available, it will execute it and parse out the version numb

[PATCH] D58069: [Sema] Mark GNU compound literal array init as an rvalue.

efriedma created this revision. efriedma added a reviewer: rsmith. Herald added a project: clang. Basically the same issue as string init, except it didn't really have any visible consequences before I removed the implicit lvalue-to-rvalue conversion from CodeGen. While I'm here, a couple minor

[PATCH] D57915: [COFF, ARM64] Remove definitions for _byteswap library functions

TomTan added a comment. In D57915#1393510 , @thakis wrote: > In D57915#1389722 , @TomTan wrote: > > > In D57915#1389560 , @lebedev.ri > > wrote: > > > > > In D57915#1389549

[PATCH] D58061: Fix a few tests that were missing ':' on CHECK lines and weren't testing anything.

This revision was automatically updated to reflect the committed changes. Closed by commit rC353744: Fix a few tests that were missing ':' on CHECK lines and weren't testing… (authored by nico, committed by ). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D580

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin added a comment. If people have opinions on this final version, please let me know. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874 ___ cfe-commits mailing list cfe-co

r353744 - Fix a few tests that were missing ':' on CHECK lines and weren't testing anything.

Author: nico Date: Mon Feb 11 12:33:22 2019 New Revision: 353744 URL: http://llvm.org/viewvc/llvm-project?rev=353744&view=rev Log: Fix a few tests that were missing ':' on CHECK lines and weren't testing anything. Found by `git grep '\/\/ CHECK-[^: ]* ' clang/test/ | grep -v RUN:`. Also tweak C

[PATCH] D58061: Fix a few tests that were missing ':' on CHECK lines and weren't testing anything.

probinson added a comment. There has been some progress recently on better FileCheck diagnosis of likely test-writing issues, although to date it mostly requires the human to ask "what is going on here?" explicitly. I can see adding a check to detect the missing-colon-on-otherwise-valid-direct

[PATCH] D58010: [CodeGen] Set construction vtable visibility after creating initializer

This revision was automatically updated to reflect the committed changes. Closed by commit rL353742: [CodeGen] Set construction vtable visibility after creating initializer (authored by phosek, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[PATCH] D57915: [COFF, ARM64] Remove definitions for _byteswap library functions

thakis added a comment. In D57915#1389722 , @TomTan wrote: > In D57915#1389560 , @lebedev.ri > wrote: > > > In D57915#1389549 , @TomTan wrote: > > > > > Added the tests back

r353742 - [CodeGen] Set construction vtable visibility after creating initializer

Author: phosek Date: Mon Feb 11 12:13:42 2019 New Revision: 353742 URL: http://llvm.org/viewvc/llvm-project?rev=353742&view=rev Log: [CodeGen] Set construction vtable visibility after creating initializer We must only set the construction vtable visibility after we create the vtable initializer,

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

This still didn't help. I can't repro the failures onhttp:// lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast , I tried Rel+Asserts and Rel-Asserts builds. Can anyone else repro this? On Mon, Feb 11, 2019 at 1:01 PM Nico Weber via cfe-commits < cfe-commits@lists.llvm.or

[PATCH] D17444: PR26672: [MSVC] Clang does not recognize "static_assert" keyword in C mode

thakis added a comment. In D17444#1393339 , @rnk wrote: > In D17444#1392823 , @thakis wrote: > > > Instead of this patch we could have an assert.h wrapper in lib/Headers that > > defines static_assert to _Static_as

[PATCH] D57915: [COFF, ARM64] Remove definitions for _byteswap library functions

This revision was automatically updated to reflect the committed changes. Closed by commit rL353740: [COFF, ARM64] Remove definitions for _byteswap library functions (authored by TomTan, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit

r353740 - [COFF, ARM64] Remove definitions for _byteswap library functions

Author: tomtan Date: Mon Feb 11 12:04:02 2019 New Revision: 353740 URL: http://llvm.org/viewvc/llvm-project?rev=353740&view=rev Log: [COFF, ARM64] Remove definitions for _byteswap library functions _byteswap_* functions are are implemented in below file as normal function from libucrt.lib and dec

[PATCH] D58067: [Analyzer] Crash fix for FindLastStoreBRVisitor

baloghadamsoftware marked 3 inline comments as done. baloghadamsoftware added a comment. I tried very hard to create a test case where we are crashing on a true positive but I did not succeed. I am not sure whether it is possible so fixing the false positive in `CallAndMessageUnInitRefArg` also

[PATCH] D58067: [Analyzer] Crash fix for FindLastStoreBRVisitor

baloghadamsoftware created this revision. baloghadamsoftware added reviewers: NoQ, dcoughlin. baloghadamsoftware added a project: clang. Herald added subscribers: gamesh411, donat.nagy, mikhail.ramalho, a.sidorin, szepet. This patch is a fix for bug 40625

[PATCH] D58061: Fix a few tests that were missing ':' on CHECK lines and weren't testing anything.

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Good catch! Is there a reasonable way to just make FileCheck itself enforce this, or does that cause too many false positives with tests that are using e.g. `x86_64` as the entire prefix?

[PATCH] D57265: [PM/CC1] Add -f[no-]split-cold-code CC1 options to toggle splitting

vsk marked an inline comment as done. vsk added a comment. > I'd prefer not adding this kind of state to PassBuilder. SplitColdCode is > soemthing that refers to the construction of one particular pipeline, not to > pipeline-building in general. It should be an argument passed down through > th

[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

erik.pilkington updated this revision to Diff 186300. erik.pilkington added a comment. Link to the GCC docs for `__builitn_object_size`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57918/new/ https://reviews.llvm.org/D57918 Files: clang/include/clang/Basic/Attr.td clang/include/c

[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

erik.pilkington marked an inline comment as done. erik.pilkington added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:987-989 +The first argument to the attribute is the type passed to +`__builtin_object_size`, and the second is the flag that the fortified fo

RE: r353718 - Make test actually test something (colons were missing)

I thin the Best Fix is probably to make FileCheck diag if a line starts with (after a few whitelisted comment chars like // and #) a check-prefix but then isn't followed by : (maybe after -NOT, -SAME, -LABEL etc). FileCheck doesn't explicitly pass over leading comment characters, it just patter

[PATCH] D58016: fixes copy constructor generation of classes containing 0-length arrays followed by exactly 1 trivial field (fixes #40682)

jbigalet added a comment. Thanks! I don't have commit rights btw Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58016/new/ https://reviews.llvm.org/D58016 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D57991: [Driver][Darwin] Emit an error when using -pg on OS without support for it.

steven_wu accepted this revision. steven_wu added a comment. This revision is now accepted and ready to land. LGTM with a suggestion to make code cleaner. Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:101 + "the clang compiler does not support -pg option on Da

[PATCH] D58016: fixes copy constructor generation of classes containing 0-length arrays followed by exactly 1 trivial field (fixes #40682)

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Ah, thanks, seems like a reasonable fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58016/new/ https://reviews.llvm.org/D58016 ___

[PATCH] D17444: PR26672: [MSVC] Clang does not recognize "static_assert" keyword in C mode

rnk added a comment. In D17444#1392823 , @thakis wrote: > Instead of this patch we could have an assert.h wrapper in lib/Headers that > defines static_assert to _Static_assert in ms mode for C files, and I suppose > that's a cleaner fix. But I hope it's

[PATCH] D56871: [AMDGPU] Require at least protected visibility for certain symbols

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

[PATCH] D58062: Support framework import/include auto-completion

sammccall added inline comments. Comment at: lib/Sema/SemaCodeComplete.cpp:8375 + auto AddFilesFromIncludeDir = [&](StringRef IncludeDir, bool IsSystem, +bool isFramework) { +bool stripFrameworkSuffix = false; nit: `IsFram

[PATCH] D58065: [analyzer] Document the frontend library

Szelethus created this revision. Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, dkrupp, rnkovacs, a.sidorin, aaron.ballman. Herald added subscribers: cfe-commits, gamesh411, donat.nagy, mikhail.ramalho, szepet, baloghadamsoftware, whisperity. Herald added a project: clang. Since I'

[PATCH] D45978: dllexport const variables must have external linkage.

aaron.ballman added inline comments. Comment at: test/Sema/dllexport-1.c:1-4 +// RUN: %clang_cc1 -triple i686-win32 -emit-llvm -fms-extensions -std=c99 < %s| FileCheck %s +// RUN: %clang_cc1 -triple x86_64-win32 -emit-llvm -fms-extensions -std=c11 < %s | FileCheck %s +// RUN: %

r353729 - Attempt to pacify bots more after r353718 and r353725

Author: nico Date: Mon Feb 11 10:01:27 2019 New Revision: 353729 URL: http://llvm.org/viewvc/llvm-project?rev=353729&view=rev Log: Attempt to pacify bots more after r353718 and r353725 Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c U

[PATCH] D56871: [AMDGPU] Require at least protected visibility for certain symbols

scott.linder added a comment. Ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56871/new/ https://reviews.llvm.org/D56871 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r353725 - Attempt to pacify bots after r353718

Author: nico Date: Mon Feb 11 09:30:25 2019 New Revision: 353725 URL: http://llvm.org/viewvc/llvm-project?rev=353725&view=rev Log: Attempt to pacify bots after r353718 Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c URL: http://llvm.

  1   2   >