[PATCH] D64666: [Sema] Enable -Wimplicit-float-conversion for integral to floating point precision loss

aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:11656 + IsListInit = + IsListInit || (isa(OrigE) && S.getLangOpts().CPlusPlus); + ziangwan wrote: > aaron.ballman wrote: > > ziangwan wrote: > > > aaron.ballman wrote: > > > >

[PATCH] D65308: [NFC][clang] Refactor getCompilationPhases()+Types.def step 3.

This revision was automatically updated to reflect the committed changes. Closed by commit rL367345: [NFC][clang] Refactor getCompilationPhases()+Types.def step 3. (authored by zer0, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

r367346 - [Parser] Lambda capture lists can start with '*'

Author: epilk Date: Tue Jul 30 12:21:20 2019 New Revision: 367346 URL: http://llvm.org/viewvc/llvm-project?rev=367346&view=rev Log: [Parser] Lambda capture lists can start with '*' Fixes llvm.org/PR42778 Modified: cfe/trunk/lib/Parse/ParseInit.cpp cfe/trunk/test/Parser/cxx0x-lambda-expre

[PATCH] D64488: [Driver] Support -fsanitize=function on Solaris/x86

ro updated this revision to Diff 212393. ro added a comment. Test i386--solaris instead of x86_64--solaris twice. Retested on x86_64-pc-solaris2.11. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64488/new/ https://reviews.llvm.org/D64488 Files: lib/Driver/Too

[PATCH] D62960: Add SVE opaque built-in types

rsandifo-arm marked 10 inline comments as done. rsandifo-arm added a comment. Thanks. I think the current version of the patch addresses all review comments so far and the last set of comments seemed positive. Does the patch look OK to land? (I was waiting until Clang 9 branched, and then was

[PATCH] D62648: [Sema][Typo] Fix assertion failure for expressions with multiple typos

rsmith added a comment. I don't think you need to change the `TreeTranform` base class to support this; `TreeTransform::TransformExpr` is an extension point which you can override from `TransformTypos` to inject the custom logic you need. But I also don't think this `TreeTransform::TransformExp

[PATCH] D63954: Add lifetime categories attributes

mgehre added a comment. I will include your latest comments into D64448 Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63954/new/ https://reviews.llvm.org/D63954 ___ cfe-commits maili

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:150 +// Log the reply. +log("<-- reply({0})", ID); +Server.onReply(std::move(ID), std::move(Result)); We want to keep logging errors (not just at verbose level).

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

xbolva00 marked 3 inline comments as done. xbolva00 added inline comments. Comment at: test/Sema/warn-int-in-bool-context.c:26 + r = a << 7; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + r = ONE << b; // expected-warning {{'<<' in boolean context; did

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

xbolva00 marked an inline comment as done. xbolva00 added inline comments. Comment at: test/Sema/warn-int-in-bool-context.c:32 + r = a ? -2 : 0; + r = a ? 3 : -2; + r = a ? 0 : TWO; // expected-warning {{'?:' with integer constants in boolean context}} jfb wr

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

jfb added inline comments. Comment at: test/Sema/warn-int-in-bool-context.c:26 + r = a << 7; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + r = ONE << b; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + xbolva00 wro

[PATCH] D62739: AMDGPU: Always emit amdgpu-flat-work-group-size

arsenm added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62739/new/ https://reviews.llvm.org/D62739 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D55895: NFC: simplify Darwin environment handling

arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55895/new/ https://reviews.llvm.org/D55895 ___ cfe-commit

[PATCH] D61466: [Rewrite][NFC] Add FIXMEs and tests for RemoveLineIfEmpty bug

jkorous added a comment. In D61466#1602928 , @jdenny wrote: > In D61466#1602917 , @jkorous wrote: > > > > > > In an inline comment, you also mentioned the alternative of replacing > `EXPECT_EQ` with `EXPECT_NE`. N

[PATCH] D64488: [Driver] Support -fsanitize=function on Solaris/x86

krytarowski added a comment. In D64488#1606758 , @ro wrote: > In D64488#1606716 , @krytarowski > wrote: > > > Something is broken between reviews. and my mailbox as I am not receiving > > any e-mails so pinging do

r367350 - [NFC] simplify Darwin environment handling

Author: jfb Date: Tue Jul 30 13:01:46 2019 New Revision: 367350 URL: http://llvm.org/viewvc/llvm-project?rev=367350&view=rev Log: [NFC] simplify Darwin environment handling The previous code detected conflicts through copy-pasta, this versions uses a 'loop'. Modified: cfe/trunk/lib/Driver/To

[PATCH] D55895: NFC: simplify Darwin environment handling

jfb updated this revision to Diff 212417. jfb added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55895/new/ https://reviews.llvm.org/D55895 Files: clang/lib/Driver/ToolChains/Darwin.cpp Index: clang/lib/Driver/ToolChains/Dar

[PATCH] D55895: NFC: simplify Darwin environment handling

This revision was automatically updated to reflect the committed changes. Closed by commit rG42c9f3c9116c: [NFC] simplify Darwin environment handling (authored by jfb). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55895/new/ https://reviews.llvm.or

r367351 - [Driver] Support -fsanitize=function on Solaris/x86

Author: ro Date: Tue Jul 30 13:04:53 2019 New Revision: 367351 URL: http://llvm.org/viewvc/llvm-project?rev=367351&view=rev Log: [Driver] Support -fsanitize=function on Solaris/x86 UBSan-Standalone-x86_64 :: TestCases/TypeCheck/Function/function.cpp currently FAILs on Solaris/x86_64: clang-9:

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

xbolva00 marked 4 inline comments as done and an inline comment as not done. xbolva00 added inline comments. Comment at: test/Sema/warn-int-in-bool-context.c:26 + r = a << 7; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + r = ONE << b; // expected-warni

[PATCH] D64488: [Driver] Support -fsanitize=function on Solaris/x86

This revision was automatically updated to reflect the committed changes. Closed by commit rL367351: [Driver] Support -fsanitize=function on Solaris/x86 (authored by ro, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://revie

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

jfb added inline comments. Comment at: test/Sema/warn-int-in-bool-context.c:26 + r = a << 7; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + r = ONE << b; // expected-warning {{'<<' in boolean context; did you mean '<'?}} + xbolva00 wro

r367353 - Revert "[NFC][clang] Refactor getCompilationPhases()+Types.def step 3."

Author: sammccall Date: Tue Jul 30 13:38:11 2019 New Revision: 367353 URL: http://llvm.org/viewvc/llvm-project?rev=367353&view=rev Log: Revert "[NFC][clang] Refactor getCompilationPhases()+Types.def step 3." This reverts commit d2254dbf21a3243233b75294ef901086199df1b9. This (unintentionally?) cha

[PATCH] D65184: [Sema] Thread Safety Analysis: Fix negative capability's LockKind representation.

aaronpuchert added inline comments. Comment at: clang/test/SemaCXX/thread-safety-annotations.h:47 +// Enable thread safety attributes only with clang. +// The attributes can be safely erased when compiling with other compilers. +#if defined(__clang__) && (!defined(SWIG))

[clang-tools-extra] r367354 - [clangd][NFC] Typo in comment

Author: jkorous Date: Tue Jul 30 13:39:39 2019 New Revision: 367354 URL: http://llvm.org/viewvc/llvm-project?rev=367354&view=rev Log: [clangd][NFC] Typo in comment Modified: clang-tools-extra/trunk/clangd/Headers.h Modified: clang-tools-extra/trunk/clangd/Headers.h URL: http://llvm.org/view

[PATCH] D65050: [SemaTemplate] Mark a function type as dependent when its parameter list contains pack expansion

cpplearner added a comment. In D65050#1606022 , @aaron.ballman wrote: > The noexcept specifier is part of the type these days, is that also handled > properly? I believe that it's properly handled in this section of `FunctionProtoType::FunctionProtoTy

[PATCH] D65337: [clangd] Disallow extraction of expression-statements.

sammccall updated this revision to Diff 212433. sammccall added a comment. Add basic test for outerImplicit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65337/new/ https://reviews.llvm.org/D65337 Files: clang-tools-extra/clangd/Selection.cpp

[PATCH] D65337: [clangd] Disallow extraction of expression-statements.

sammccall updated this revision to Diff 212432. sammccall marked 2 inline comments as done. sammccall added a comment. Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65337/new/ https://reviews.llvm.org/D65337 Files: clang

[PATCH] D65337: [clangd] Disallow extraction of expression-statements.

sammccall marked 6 inline comments as done. sammccall added a comment. In D65337#1604324 , @SureYeaah wrote: > What was the bug in getCallExpr() ? It could find calls where the DeclRef was an arbitrary subexpression of the callee, not exactly the callee

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

xbolva00 marked 2 inline comments as done and an inline comment as not done. xbolva00 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; -

[PATCH] D63907: [clang-scan-deps] Implementation of dependency scanner over minimized sources

arphaman updated this revision to Diff 212435. arphaman marked 7 inline comments as done. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63907/new/ https://reviews.llvm.org/D63907 Files: clang/include/clang/Basic/FileManager.h clang/include/clang/Tooling/Depend

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

jfb 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: > jfb wrote: > > aaron.ballman wrote: > > > xbolva00 wrote

[PATCH] D63139: [Diagnostics] Implement -Wswitch-unreachable

xbolva00 accepted this revision. xbolva00 added a comment. This revision is now accepted and ready to land. Since motivation examples are already handled by -Wunreachable-code, I will close this revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63139/new/ https://reviews.llvm.org

[PATCH] D63907: [clang-scan-deps] Implementation of dependency scanner over minimized sources

arphaman added inline comments. Comment at: clang/include/clang/Tooling/DependencyScanning/DependencyScanningFilesystem.h:1 +//===- DependencyScanningFilesystem.h - clang-scan-deps fs ===---*- C++ -*-===// +// aganea wrote: > General comment for this file and t

[PATCH] D65184: [Sema] Thread Safety Analysis: Fix negative capability's LockKind representation.

ziangwan marked 2 inline comments as done. ziangwan added inline comments. Comment at: clang/test/SemaCXX/thread-safety-annotations.h:47 +// Enable thread safety attributes only with clang. +// The attributes can be safely erased when compiling with other compilers. +#if defined(

[PATCH] D64448: gsl::Owner/gsl::Pointer: Add implicit annotations for some std types

mgehre updated this revision to Diff 212439. mgehre marked 6 inline comments as done. mgehre added a comment. - Fix comments - Add Pointer via typedef on ClassTemplateDecl Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64448/new/ https://reviews.llv

[PATCH] D64448: gsl::Owner/gsl::Pointer: Add implicit annotations for some std types

mgehre added inline comments. Comment at: clang/include/clang/Sema/Sema.h:6097 + + /// Add [[gsl::Owner]] and [[gsl::Pointer]] attributes for std:: types. + void addDefaultGslPointerAttribute(TypedefNameDecl *TD); gribozavr wrote: > It seems like this function

[PATCH] D65239: [analyzer] RangeConstraintManager: Apply constraint ranges of bitwise operations

Charusso updated this revision to Diff 212440. Charusso marked 6 inline comments as done. Charusso added a comment. - Fix. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65239/new/ https://reviews.llvm.org/D65239 Files: clang/include/clang/AST/Expr.h clang/include/clang/StaticAnaly

[PATCH] D65239: [analyzer] RangeConstraintManager: Apply constraint ranges of bitwise operations

Charusso added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RangeConstraintManager.cpp:496 + // as a bitwise operation result could be null. + if (RS.getConcreteValue() && RS.getConcreteValue()->getExtValue() == 0) +return State; NoQ wrote: >

[PATCH] D64448: gsl::Owner/gsl::Pointer: Add implicit annotations for some std types

mgehre updated this revision to Diff 212441. mgehre added a comment. - Add missing check Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64448/new/ https://reviews.llvm.org/D64448 Files: clang/include/clang/Basic/AttrDocs.td clang/include/clang/

[PATCH] D65184: [Sema] Thread Safety Analysis: Fix negative capability's LockKind representation.

aaronpuchert added a comment. The primary purpose of Thread Safety Analysis is to make sure that accesses to shared resources are protected. That's why we only track whether a lock is available by default (i.e. without -Wthread-safety-negative), locks that we don't know anything about are assum

[PATCH] D63907: [clang-scan-deps] Implementation of dependency scanner over minimized sources

arphaman updated this revision to Diff 212442. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63907/new/ https://reviews.llvm.org/D63907 Files: clang/include/clang/Basic/FileManager.h clang/include/clang/Tooling/DependencyScanning/DependencyScanningFilesystem.h clang/include/clang/T

[PATCH] D65184: [Sema] Thread Safety Analysis: Fix negative capability's LockKind representation.

aaronpuchert added inline comments. Comment at: clang/test/SemaCXX/warn-thread-safety-negative.cpp:135-140 + if (condition) { +assertNotHeld(); // expected-warning {{mutex '!mu' is acquired exclusively and shared in the same scope}} + } else { +mu.Lock(); +mu.Unloc

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

xbolva00 marked 2 inline comments as done. xbolva00 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; jfb wrote: > xbolva00 wrot

[PATCH] D64448: gsl::Owner/gsl::Pointer: Add implicit annotations for some std types

mgehre updated this revision to Diff 212446. mgehre added a comment. - Fix crash Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64448/new/ https://reviews.llvm.org/D64448 Files: clang/include/clang/Basic/AttrDocs.td clang/include/clang/Sema/Sem

[clang-tools-extra] r367361 - [clangd][NFC] Fix typo in comment

Author: jkorous Date: Tue Jul 30 15:18:04 2019 New Revision: 367361 URL: http://llvm.org/viewvc/llvm-project?rev=367361&view=rev Log: [clangd][NFC] Fix typo in comment Modified: clang-tools-extra/trunk/clangd/ClangdUnit.cpp Modified: clang-tools-extra/trunk/clangd/ClangdUnit.cpp URL: http:/

[PATCH] D65481: NFCI: Simplify SourceManager::translateFile by removing code path that should never be taken

arphaman created this revision. arphaman added reviewers: rsmith, bruno, Bigcheese. Herald added subscribers: dexonsmith, jkorous. Herald added a project: clang. I noticed that `SourceManager::translateFile` has code that doesn't really make sense. In particular, if it fails to find a `FileID` by

[PATCH] D65484: [analyzer][NFC] Refactoring BugReporter.cpp P5.: Compact mile long function invocations into objects

Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, baloghadamsoftware, Charusso, dcoughlin, rnkovacs. Szelethus added a project: clang. Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity. This patch is sca

[PATCH] D65483: [clang-format] Add link to source code in file definitions

DiegoAstiazaran created this revision. DiegoAstiazaran added reviewers: juliehockett, jakehehrlich, lebedev.ri. DiegoAstiazaran added a project: clang-tools-extra. Herald added subscribers: kadircet, arphaman. Two command line options have been added to clang-doc. --repository= - URL of r

[PATCH] D65485: Revert "[NFC][clang] Refactor getCompilationPhases()+Types.def step 3."

sammccall created this revision. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous. Herald added a project: clang. This reverts commit d2254dbf21a3243233b75294ef901086199df1b9 . This (unintentionally?) changed b

[PATCH] D65486: [clangd] Ignore semicolons, whitespace, and comments in SelectionTree.

sammccall created this revision. sammccall added reviewers: SureYeaah, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay. Herald added a project: clang. Whitespace and comments are a clear bugfix: selecting some comments/space near a statement doesn't mean you're selectin

[PATCH] D65487: [analyzer][NFC] Refactoring BugReporter.cpp P6.: Completely get rid of interestingness propagation

Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, rnkovacs, baloghadamsoftware, Charusso, dcoughlin. Szelethus added a project: clang. Herald added subscribers: cfe-commits, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity. Do you remember t

[PATCH] D65485: Revert "[NFC][clang] Refactor getCompilationPhases()+Types.def step 3."

sammccall abandoned this revision. sammccall added a comment. Please ignore this, I messed up with git/arc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65485/new/ https://reviews.llvm.org/D65485 ___ c

[PATCH] D65486: [clangd] Ignore semicolons, whitespace, and comments in SelectionTree.

sammccall added a comment. It occurs to me that `claim` is `O(node_tokens + log total_tokens)` which is bad when the nodes are large. Indeed for an input like `namespace { namespace { namespace { ... } } }` time is quadratic. I think this is probably fine in practice. Against adversarial input

r367367 - [Sema] Actually map a variable template specialization from pattern to instantiation

Author: epilk Date: Tue Jul 30 16:38:18 2019 New Revision: 367367 URL: http://llvm.org/viewvc/llvm-project?rev=367367&view=rev Log: [Sema] Actually map a variable template specialization from pattern to instantiation We were previously just using a specialization in the class template instead of

[PATCH] D65359: [Sema] Map from a variable template specialization in a pattern to a variable template specialization in an instantiation properly

This revision was automatically updated to reflect the committed changes. Closed by commit rL367367: [Sema] Actually map a variable template specialization from pattern to… (authored by epilk, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[PATCH] D65022: [Sema] Always instantiate the initializer of a variable template with undeduced type (8.0 regression)

erik.pilkington updated this revision to Diff 212470. erik.pilkington added a comment. Rebase on top of r367367. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65022/new/ https://reviews.llvm.org/D65022 Files: clang/lib/Sema/SemaExpr.cpp clang/lib/Sema/SemaExprMember.cpp clang/tes

r367371 - NFCI, optimize layout of FileEntry

Author: arphaman Date: Tue Jul 30 17:12:00 2019 New Revision: 367371 URL: http://llvm.org/viewvc/llvm-project?rev=367371&view=rev Log: NFCI, optimize layout of FileEntry The reordering of the UID field makes the size of a FileEntry 8 bytes smaller on 64bit platforms. Modified: cfe/trunk/incl

[PATCH] D65493: Modernize atomic detection and usage

jfb created this revision. jfb added reviewers: rnk, Bigcheese, __simt__. Herald added subscribers: llvm-commits, lldb-commits, cfe-commits, kadircet, arphaman, dexonsmith, jkorous, hiraditya, mgorny. Herald added projects: clang, LLDB, LLVM. Some of the cmake checks are obsolete and make bootstr

[PATCH] D65493: Modernize atomic detection and usage

jfb updated this revision to Diff 212479. jfb added a comment. - Remove from cmake Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65493/new/ https://reviews.llvm.org/D65493 Files: clang-tools-extra/clangd/CMakeLists.txt lldb/source/Utility/CMak

[PATCH] D65300: [clang] [CodeGen] clang-misexpect prototype for compiler warnings

paulkirth updated this revision to Diff 212492. paulkirth added a comment. Update diff to have proper context on Phabricator Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65300/new/ https://reviews.llvm.org/D65300 Files: clang/include/clang/Basi

[PATCH] D65483: [clang-doc] Add link to source code in file definitions

Eugene.Zelenko added a comment. It'll be reasonable to mention new command-line arguments in documentation and Release Notes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65483/new/ https://reviews.llvm.org/D65483 ___ cfe-commits mailing

[PATCH] D65300: [clang] [CodeGen] clang-misexpect prototype for compiler warnings

phosek added inline comments. Comment at: clang/lib/CodeGen/MisExpect.cpp:29 + +void DebugPrintMisExpectSwitchInfo(SmallVector *SwitchWeights, + llvm::DenseMap *CaseMap); It seems like `DebugPrintMisExpectSwitchInfo` and `EmitMis

[PATCH] D65300: [clang] [CodeGen] clang-misexpect prototype for compiler warnings

leonardchan accepted this revision. leonardchan added a comment. This revision is now accepted and ready to land. LGTM but I'd wait a day to see if anyone else has comments they'd like to add before landing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[clang-tools-extra] r367383 - [NFC] Remove LLVM_ALIGNAS

Author: jfb Date: Tue Jul 30 20:22:08 2019 New Revision: 367383 URL: http://llvm.org/viewvc/llvm-project?rev=367383&view=rev Log: [NFC] Remove LLVM_ALIGNAS Summary: The minimum compilers support all have alignas, and we don't use LLVM_ALIGNAS anywhere anymore. This also removes an MSVC diagnosti

r367383 - [NFC] Remove LLVM_ALIGNAS

Author: jfb Date: Tue Jul 30 20:22:08 2019 New Revision: 367383 URL: http://llvm.org/viewvc/llvm-project?rev=367383&view=rev Log: [NFC] Remove LLVM_ALIGNAS Summary: The minimum compilers support all have alignas, and we don't use LLVM_ALIGNAS anywhere anymore. This also removes an MSVC diagnosti

[PATCH] D65458: [NFC] Remove LLVM_ALIGNAS

This revision was automatically updated to reflect the committed changes. Closed by commit rL367383: [NFC] Remove LLVM_ALIGNAS (authored by jfb, committed by ). Herald added a subscriber: kristina. Changed prior to commit: https://reviews.llvm.org/D65458?vs=212368&id=212496#toc Repository: r

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

jdoerfert updated this revision to Diff 212500. jdoerfert added a comment. Fix spelling in tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58091/new/ https://reviews.llvm.org/D58091 Files: clang/include/clang/Basic/DiagnosticSemaKinds.td c

r367387 - [Fix] Customize warnings for missing built-in types

Author: jdoerfert Date: Tue Jul 30 22:16:38 2019 New Revision: 367387 URL: http://llvm.org/viewvc/llvm-project?rev=367387&view=rev Log: [Fix] Customize warnings for missing built-in types If we detect a built-in declaration for which we cannot derive a type matching the pattern in the Builtins.de

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

This revision was automatically updated to reflect the committed changes. Closed by commit rL367387: [Fix] Customize warnings for missing built-in types (authored by jdoerfert, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https:

[PATCH] D65445: [CrossTU] Handle case when no USR could be generated during Decl search.

balazske added a comment. It looks like that the problem can happen when the anonymous union is in any `DeclContext` and for CTU import the import of a variable is requested and that variable is in a related `DeclContext` (it can be at upper or lower level). (See code of `findDefInDeclContext`:

<    1   2