r339409 - Fix size calculation from r339380

2018-08-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Aug 9 18:30:10 2018 New Revision: 339409 URL: http://llvm.org/viewvc/llvm-project?rev=339409&view=rev Log: Fix size calculation from r339380 r339380 changed the trailing types of ParsedAttr to use llvm::TrailingObjects. However, it did not copy over one of the size attri

r340357 - Add space to TemplateArgument dump

2018-08-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Aug 21 15:55:26 2018 New Revision: 340357 URL: http://llvm.org/viewvc/llvm-project?rev=340357&view=rev Log: Add space to TemplateArgument dump Add a missing space when dumping a template argument which is a template expansion. Found during debugging so no test. Modified

r332310 - Enable control flow pruning of float overflow warnings.

2018-05-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon May 14 16:21:48 2018 New Revision: 332310 URL: http://llvm.org/viewvc/llvm-project?rev=332310&view=rev Log: Enable control flow pruning of float overflow warnings. Like other conversion warnings, allow float overflow warnings to be disabled in known dead paths of template

r333471 - Check pointer null-ness before dereferencing it.

2018-05-29 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue May 29 15:43:00 2018 New Revision: 333471 URL: http://llvm.org/viewvc/llvm-project?rev=333471&view=rev Log: Check pointer null-ness before dereferencing it. -Warc-repeated-use-of-weak may trigger a segmentation fault when the Decl being checked is outside of a function sc

r333486 - [ODRHash] Support FunctionTemplateDecl in records.

2018-05-29 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue May 29 18:12:26 2018 New Revision: 333486 URL: http://llvm.org/viewvc/llvm-project?rev=333486&view=rev Log: [ODRHash] Support FunctionTemplateDecl in records. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td cfe/trunk/lib/AST/ODRHash.cpp

r314581 - [ODRHash] Add base classes to hashing CXXRecordDecl.

2017-09-29 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 29 19:19:17 2017 New Revision: 314581 URL: http://llvm.org/viewvc/llvm-project?rev=314581&view=rev Log: [ODRHash] Add base classes to hashing CXXRecordDecl. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td cfe/trunk/include/clang/Seriali

r341013 - Ensure canonical type is actually canonical.

2018-08-29 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Aug 29 18:57:52 2018 New Revision: 341013 URL: http://llvm.org/viewvc/llvm-project?rev=341013&view=rev Log: Ensure canonical type is actually canonical. ASTContext::applyObjCProtocolQualifiers will return a canonical type when given a canonical type and an array of canoni

r341421 - [ODRHash] Extend hash to support all Type's.

2018-09-04 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Sep 4 15:53:19 2018 New Revision: 341421 URL: http://llvm.org/viewvc/llvm-project?rev=341421&view=rev Log: [ODRHash] Extend hash to support all Type's. Added: cfe/trunk/test/Modules/odr_hash-gnu.cpp cfe/trunk/test/Modules/odr_hash-vector.cpp cfe/trunk/test/Mo

r341496 - Add triple to test case.

2018-09-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Sep 5 14:55:09 2018 New Revision: 341496 URL: http://llvm.org/viewvc/llvm-project?rev=341496&view=rev Log: Add triple to test case. This test uses enums, which have different behavior when targeting different systems. Specifying a triple will give predictable behavior t

Re: r341421 - [ODRHash] Extend hash to support all Type's.

2018-09-05 Thread Richard Trieu via cfe-commits
LLVM :: CodeGen/AMDGPU/mubuf-legalize-operands.mir > > Please have a look? > The builder was already red and did not send notifications on this. > > Thanks > > Galina > > On Tue, Sep 4, 2018 at 3:54 PM Richard Trieu via cfe-commits < > cfe-commits@lists.llvm.org> wr

r341498 - Remove unnecessary options from test RUN lines.

2018-09-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Sep 5 15:14:46 2018 New Revision: 341498 URL: http://llvm.org/viewvc/llvm-project?rev=341498&view=rev Log: Remove unnecessary options from test RUN lines. These tests do not check the color printing, so color options should not be used when running them. Modified: c

r342199 - [ODRHash] Fix early exit that skipped code.

2018-09-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Sep 13 18:15:28 2018 New Revision: 342199 URL: http://llvm.org/viewvc/llvm-project?rev=342199&view=rev Log: [ODRHash] Fix early exit that skipped code. There is a bit of code at the end of AddDeclaration that should be run on every exit of the function. However, there wa

r337978 - [ODRHash] Support hashing enums.

2018-07-25 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jul 25 15:52:05 2018 New Revision: 337978 URL: http://llvm.org/viewvc/llvm-project?rev=337978&view=rev Log: [ODRHash] Support hashing enums. Modified: cfe/trunk/include/clang/AST/Decl.h cfe/trunk/include/clang/AST/ODRHash.h cfe/trunk/include/clang/Basic/Diagno

r324308 - Fix crash on invalid.

2018-02-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Feb 5 18:58:21 2018 New Revision: 324308 URL: http://llvm.org/viewvc/llvm-project?rev=324308&view=rev Log: Fix crash on invalid. Don't call a method when the pointer is null. Modified: cfe/trunk/lib/Sema/SemaExpr.cpp cfe/trunk/test/SemaCXX/lambda-expressions.cpp

r325040 - Update StmtProfile.cpp to handle zero template arguments.

2018-02-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Feb 13 11:53:40 2018 New Revision: 325040 URL: http://llvm.org/viewvc/llvm-project?rev=325040&view=rev Log: Update StmtProfile.cpp to handle zero template arguments. Treat having no templates arguments differently than having zero template arguments when profiling. Modif

Re: r324308 - Fix crash on invalid.

2018-02-21 Thread Richard Trieu via cfe-commits
Hi Hans, If there's still time for rc3, I'd like to get this crash fix in. This adds a null check to prevent a crash on invalid. Richard On Mon, Feb 5, 2018 at 6:58 PM, Richard Trieu via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rtrieu > Date: Mon Feb

r325741 - [ODRHash] Handle some template weirdness.

2018-02-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Feb 21 21:32:25 2018 New Revision: 325741 URL: http://llvm.org/viewvc/llvm-project?rev=325741&view=rev Log: [ODRHash] Handle some template weirdness. Build the index off of DeclarationName instead of Decl pointers. When finding an UnresolvedLookupExprClass, hash it as if

r325742 - [ODRHash] Fix hashing for friend functions.

2018-02-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Feb 21 21:50:29 2018 New Revision: 325742 URL: http://llvm.org/viewvc/llvm-project?rev=325742&view=rev Log: [ODRHash] Fix hashing for friend functions. When hashing a templated function, use the hash of the function it was instantiated from. Added: cfe/trunk/test/Mod

Re: r324308 - Fix crash on invalid.

2018-02-22 Thread Richard Trieu via cfe-commits
adds > > a null check to prevent a crash on invalid. > > > > Richard > > > > On Mon, Feb 5, 2018 at 6:58 PM, Richard Trieu via cfe-commits > > wrote: > >> > >> Author: rtrieu > >> Date: Mon Feb 5 18:58:21 2018 > >> New Rev

r303932 - Make test/Driver/baremetal.cpp work when output directory isn't named 'bin'.

2017-05-25 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu May 25 18:03:08 2017 New Revision: 303932 URL: http://llvm.org/viewvc/llvm-project?rev=303932&view=rev Log: Make test/Driver/baremetal.cpp work when output directory isn't named 'bin'. Modified: cfe/trunk/test/Driver/baremetal.cpp Modified: cfe/trunk/test/Driver/bar

r303934 - "*" => "+" to avoid matching on empty string.

2017-05-25 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu May 25 18:25:36 2017 New Revision: 303934 URL: http://llvm.org/viewvc/llvm-project?rev=303934&view=rev Log: "*" => "+" to avoid matching on empty string. Modified: cfe/trunk/test/Driver/baremetal.cpp Modified: cfe/trunk/test/Driver/baremetal.cpp URL: http://llvm.org

Re: r303934 - "*" => "+" to avoid matching on empty string.

2017-05-29 Thread Richard Trieu via cfe-commits
eparator would match the "\" before clang. On Mon, May 29, 2017 at 11:18 AM, David Blaikie wrote: > Why would matching on an empty string be bad in this case? > > On Thu, May 25, 2017 at 4:25 PM Richard Trieu via cfe-commits < > cfe-commits@lists.llvm.org> wrote: >

r304261 - [ODRHash] Support TemplateSpecializationType

2017-05-30 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue May 30 19:31:58 2017 New Revision: 304261 URL: http://llvm.org/viewvc/llvm-project?rev=304261&view=rev Log: [ODRHash] Support TemplateSpecializationType Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/trunk/lib/AST/ODRHas

r304519 - Minor fixes to for-loop warning.

2017-06-01 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Jun 1 23:24:46 2017 New Revision: 304519 URL: http://llvm.org/viewvc/llvm-project?rev=304519&view=rev Log: Minor fixes to for-loop warning. The warning for unchanged loop variables outputted a diagnostic that was dependent on iteration order from a pointer set, which is

r304592 - [ODRHash] Add support for TemplateArgument types.

2017-06-02 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 2 15:35:29 2017 New Revision: 304592 URL: http://llvm.org/viewvc/llvm-project?rev=304592&view=rev Log: [ODRHash] Add support for TemplateArgument types. Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/trunk/lib/AST/O

r304618 - Revert r304592

2017-06-02 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 2 19:11:23 2017 New Revision: 304618 URL: http://llvm.org/viewvc/llvm-project?rev=304618&view=rev Log: Revert r304592 r304592 - [ODRHash] Add support for TemplateArgument types. Possibly causing one of the errors in modules build bot. Modified: cfe/trunk/lib/AST

r304956 - [ODRHash] Change the fall-back diagnostic error.

2017-06-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jun 7 19:56:21 2017 New Revision: 304956 URL: http://llvm.org/viewvc/llvm-project?rev=304956&view=rev Log: [ODRHash] Change the fall-back diagnostic error. Provide a little more information when a ODR violation is detected, but the specific error could not be diagnosed.

Re: r304956 - [ODRHash] Change the fall-back diagnostic error.

2017-06-07 Thread Richard Trieu via cfe-commits
. On Wed, Jun 7, 2017 at 6:05 PM, Richard Smith wrote: > On 7 June 2017 at 17:56, Richard Trieu via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: rtrieu >> Date: Wed Jun 7 19:56:21 2017 >> New Revision: 304956 >> >> URL: http://llvm.

r304962 - [ODRHash] Make diagnostic message more readable.

2017-06-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jun 7 23:47:29 2017 New Revision: 304962 URL: http://llvm.org/viewvc/llvm-project?rev=304962&view=rev Log: [ODRHash] Make diagnostic message more readable. Change the diagnostic message from r304956 to be less confusing by reordering the flow of information. Modified:

Re: r304956 - [ODRHash] Change the fall-back diagnostic error.

2017-06-07 Thread Richard Trieu via cfe-commits
s diagnostic or > the one below it. Except that the hasher isn't complete yet and some > things slip through. Once things are more stable, these should be replaced > with llvm_unreachable instead. > > On Wed, Jun 7, 2017 at 6:05 PM, Richard Smith > wrote: > >> On

r305104 - [ODRHash] Skip inline namespaces when hashing.

2017-06-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 9 15:11:51 2017 New Revision: 305104 URL: http://llvm.org/viewvc/llvm-project?rev=305104&view=rev Log: [ODRHash] Skip inline namespaces when hashing. Speculatively try to fix the underlying issue from r304592, of underlying types being confused when inline namespaces

r305110 - [ODRHash] Add support for TemplateArgument types.

2017-06-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 9 16:00:10 2017 New Revision: 305110 URL: http://llvm.org/viewvc/llvm-project?rev=305110&view=rev Log: [ODRHash] Add support for TemplateArgument types. Recommit r304592 that was reverted in r304618. r305104 should have fixed the issue. Modified: cfe/trunk/lib/

r305130 - Revert r305110 to fix buildbot

2017-06-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 9 18:03:40 2017 New Revision: 305130 URL: http://llvm.org/viewvc/llvm-project?rev=305130&view=rev Log: Revert r305110 to fix buildbot Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/trunk/lib/AST/ODRHash.cpp URL: ht

r305238 - [ODRHash] Add diagnostic messages for typedef and type alias.

2017-06-12 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Jun 12 16:58:22 2017 New Revision: 305238 URL: http://llvm.org/viewvc/llvm-project?rev=305238&view=rev Log: [ODRHash] Add diagnostic messages for typedef and type alias. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td cfe/trunk/lib/Serializ

r305328 - [ODRHash] Add TemplateArgument kind to hash.

2017-06-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Jun 13 17:21:18 2017 New Revision: 305328 URL: http://llvm.org/viewvc/llvm-project?rev=305328&view=rev Log: [ODRHash] Add TemplateArgument kind to hash. Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/trunk/lib/AST/ODRHas

r305360 - [ODRHash] Hash Expr for TemplateArgument::Expression

2017-06-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Jun 13 20:28:00 2017 New Revision: 305360 URL: http://llvm.org/viewvc/llvm-project?rev=305360&view=rev Log: [ODRHash] Hash Expr for TemplateArgument::Expression Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/trunk/lib/AS

r305361 - [ODRHash] Hash Template and TemplateExpansion in TemplateArgument.

2017-06-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Jun 13 22:17:26 2017 New Revision: 305361 URL: http://llvm.org/viewvc/llvm-project?rev=305361&view=rev Log: [ODRHash] Hash Template and TemplateExpansion in TemplateArgument. Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cf

r305362 - [ODRHash] Remove debugging code from r305361

2017-06-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Jun 13 22:19:58 2017 New Revision: 305362 URL: http://llvm.org/viewvc/llvm-project?rev=305362&view=rev Log: [ODRHash] Remove debugging code from r305361 Modified: cfe/trunk/lib/AST/ODRHash.cpp Modified: cfe/trunk/lib/AST/ODRHash.cpp URL: http://llvm.org/viewvc/llvm-

r305440 - [ODRHash] Hash TemplateArgument::Pack and TemplateTypeParmType

2017-06-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jun 14 20:35:06 2017 New Revision: 305440 URL: http://llvm.org/viewvc/llvm-project?rev=305440&view=rev Log: [ODRHash] Hash TemplateArgument::Pack and TemplateTypeParmType Modified: cfe/trunk/lib/AST/ODRHash.cpp cfe/trunk/test/Modules/odr_hash.cpp Modified: cfe/tr

r305543 - [ODRHash] Hash VarDecl members.

2017-06-15 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Jun 15 21:44:29 2017 New Revision: 305543 URL: http://llvm.org/viewvc/llvm-project?rev=305543&view=rev Log: [ODRHash] Hash VarDecl members. These VarDecl's are static data members of classes. Since the initializers are also hashed, this also provides checking for default

r305872 - [ODRHash] Supply more information when generic error message is emitted.

2017-06-20 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Jun 20 20:43:13 2017 New Revision: 305872 URL: http://llvm.org/viewvc/llvm-project?rev=305872&view=rev Log: [ODRHash] Supply more information when generic error message is emitted. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td cfe/trunk/l

r321319 - [ODRHash] Canonicalize Decl's before processing.

2017-12-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Dec 21 14:38:29 2017 New Revision: 321319 URL: http://llvm.org/viewvc/llvm-project?rev=321319&view=rev Log: [ODRHash] Canonicalize Decl's before processing. Canonicalizing the Decl before processing it as part of the hash should reduce issues with non-canonical types show

r321395 - [ODRHash] Support ODR violation detection in functions.

2017-12-22 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 22 16:41:01 2017 New Revision: 321395 URL: http://llvm.org/viewvc/llvm-project?rev=321395&view=rev Log: [ODRHash] Support ODR violation detection in functions. Extend the hashing to functions, which allows detection of function definition mismatches across modules. T

r321396 - [ODRHash] Disable hashing on methods.

2017-12-22 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 22 17:35:32 2017 New Revision: 321396 URL: http://llvm.org/viewvc/llvm-project?rev=321396&view=rev Log: [ODRHash] Disable hashing on methods. Turn off hashing for class methods, but leave it on for other functions. This should get the buildbot to green for the time b

Re: r321395 - [ODRHash] Support ODR violation detection in functions.

2018-01-03 Thread Richard Trieu via cfe-commits
> 2 errors generated. > > I'm not sure how to act on this, because it looks like the error is saying > that a definition in 'std.list' is incompatible with itself. > > I've temporarily disabled building with modules enabled on the bot. Could > you take a look? &g

r321924 - Test case for r321396

2018-01-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jan 5 19:20:59 2018 New Revision: 321924 URL: http://llvm.org/viewvc/llvm-project?rev=321924&view=rev Log: Test case for r321396 Any hashing for methods should be able to compile this test case without emitting an error. Since the class and method come from the same hea

r327593 - [CFG] Allow CallExpr's to be looked up in CFG's

2018-03-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Mar 14 17:09:26 2018 New Revision: 327593 URL: http://llvm.org/viewvc/llvm-project?rev=327593&view=rev Log: [CFG] Allow CallExpr's to be looked up in CFG's r327343 changed the handling for CallExpr in a CFG, which prevented lookups for CallExpr while other Stmt kinds stil

r327598 - Refactoring code around move/copy initialization. NFC.

2018-03-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Mar 14 20:00:55 2018 New Revision: 327598 URL: http://llvm.org/viewvc/llvm-project?rev=327598&view=rev Log: Refactoring code around move/copy initialization. NFC. Use an enum parameter instead of a bool for more control on how the copy elision functions work. Extract th

r334151 - [ODRHash] Adjust info stored for FunctionTemplateDecl.

2018-06-06 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jun 6 17:20:58 2018 New Revision: 334151 URL: http://llvm.org/viewvc/llvm-project?rev=334151&view=rev Log: [ODRHash] Adjust info stored for FunctionTemplateDecl. Avoid storing information for definitions since those can be out-of-line and vary between modules even when t

r334160 - Change return value of trivial visibility check.

2018-06-06 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jun 6 20:20:30 2018 New Revision: 334160 URL: http://llvm.org/viewvc/llvm-project?rev=334160&view=rev Log: Change return value of trivial visibility check. Previous, if no Decl's were checked, visibility was set to false. Switch it so that in cases of no Decl's, return

r344828 - Make -Wfor-loop-analysis work with C++17

2018-10-19 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Oct 19 19:15:58 2018 New Revision: 344828 URL: http://llvm.org/viewvc/llvm-project?rev=344828&view=rev Log: Make -Wfor-loop-analysis work with C++17 For now, disable the "variable in loop condition not modified" warning to not be emitted when there is a structured binding

r344987 - [CodeGen] Attach InlineHint to more functions

2018-10-22 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Oct 22 18:26:28 2018 New Revision: 344987 URL: http://llvm.org/viewvc/llvm-project?rev=344987&view=rev Log: [CodeGen] Attach InlineHint to more functions For instantiated functions, search the template pattern to see if it marked inline to determine if InlineHint attribut

r345111 - [Sema] Fix -Wcomma in dependent context

2018-10-23 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Oct 23 19:07:41 2018 New Revision: 345111 URL: http://llvm.org/viewvc/llvm-project?rev=345111&view=rev Log: [Sema] Fix -Wcomma in dependent context When there is a dependent type inside a cast, the CastKind becomes CK_Dependent instead of CK_ToVoid. This fix will check t

r345228 - [Sema] Fix -Wcomma for C89

2018-10-24 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Oct 24 18:08:00 2018 New Revision: 345228 URL: http://llvm.org/viewvc/llvm-project?rev=345228&view=rev Log: [Sema] Fix -Wcomma for C89 There is a small difference in the scope flags for C89 versus the other C/C++ dialects. This change ensures that the -Wcomma warning won

r345669 - Silence unused variable warnings. NFC

2018-10-30 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Oct 30 16:01:15 2018 New Revision: 345669 URL: http://llvm.org/viewvc/llvm-project?rev=345669&view=rev Log: Silence unused variable warnings. NFC Modified: cfe/trunk/lib/CodeGen/CGExprScalar.cpp Modified: cfe/trunk/lib/CodeGen/CGExprScalar.cpp URL: http://llvm.org/

r345691 - Revert r345676 due to test failure.

2018-10-30 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Oct 30 19:10:51 2018 New Revision: 345691 URL: http://llvm.org/viewvc/llvm-project?rev=345691&view=rev Log: Revert r345676 due to test failure. This was causing CodeGen/mingw-long-double.c to start failing. Removed: cfe/trunk/test/CodeGen/win64-i128.c Modified: c

Re: r345676 - [Win64] Handle passing i128 by value

2018-10-30 Thread Richard Trieu via cfe-commits
I have reverted this in r345691 because it caused test CodeGen/mingw-long-double.c to start failing. Command Output (stderr): -- /usr/local/google/clang/install/llvm/tools/clang/test/CodeGen/mingw-long-double.c:36:11: error: MSC64: expected string not found in input // MSC64: define dso_local doub

r347726 - [diagtool] Remove unneeded header includes.

2018-11-27 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Nov 27 19:59:35 2018 New Revision: 347726 URL: http://llvm.org/viewvc/llvm-project?rev=347726&view=rev Log: [diagtool] Remove unneeded header includes. Modified: cfe/trunk/tools/diagtool/ListWarnings.cpp cfe/trunk/tools/diagtool/TreeView.cpp Modified: cfe/trunk/t

r347727 - [CodeGen] Fix included headers.

2018-11-27 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Nov 27 20:14:29 2018 New Revision: 347727 URL: http://llvm.org/viewvc/llvm-project?rev=347727&view=rev Log: [CodeGen] Fix included headers. Remove the included Parse header because CodeGen should not depend on Parse. Instead, include the Lex headers that it needs instead.

r347728 - Move LoopHint.h from Sema to Parse

2018-11-27 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Nov 27 20:36:31 2018 New Revision: 347728 URL: http://llvm.org/viewvc/llvm-project?rev=347728&view=rev Log: Move LoopHint.h from Sema to Parse struct LoopHint was only used within Parse and not in any of the Sema or Codegen files. In the non-Parse files where it was incl

r348238 - Remove unnecessary include.

2018-12-03 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Dec 3 20:53:18 2018 New Revision: 348238 URL: http://llvm.org/viewvc/llvm-project?rev=348238&view=rev Log: Remove unnecessary include. Modified: cfe/trunk/lib/CodeGen/CodeGenTypes.h Modified: cfe/trunk/lib/CodeGen/CodeGenTypes.h URL: http://llvm.org/viewvc/llvm-pro

r348458 - Remove CodeGen dependencies on Sema.

2018-12-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Dec 5 22:12:20 2018 New Revision: 348458 URL: http://llvm.org/viewvc/llvm-project?rev=348458&view=rev Log: Remove CodeGen dependencies on Sema. Move diagnostics from Sema to Frontend (or Common) so that CodeGen no longer needs to include the Sema diagnostic IDs. Modifie

r348459 - Remove unnecessary include.

2018-12-05 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Dec 5 22:32:40 2018 New Revision: 348459 URL: http://llvm.org/viewvc/llvm-project?rev=348459&view=rev Log: Remove unnecessary include. Modified: cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp Modified: cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperatio

r348685 - Move diagnostic enums into Basic.

2018-12-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 7 21:05:03 2018 New Revision: 348685 URL: http://llvm.org/viewvc/llvm-project?rev=348685&view=rev Log: Move diagnostic enums into Basic. Move enums from */*Diagnostic.h to Basic/Diagnostic*.h. Basic/AllDiagnostics.h needs all the enums and moving the sources to Basi

r348827 - Move CodeGenOptions from Frontend to Basic

2018-12-10 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Dec 10 19:18:39 2018 New Revision: 348827 URL: http://llvm.org/viewvc/llvm-project?rev=348827&view=rev Log: Move CodeGenOptions from Frontend to Basic Basic uses CodeGenOptions and should not depend on Frontend. Added: cfe/trunk/include/clang/Basic/CodeGenOptions.def

r356231 - Remove an assert in template pack deduction during nested instantiation.

2019-03-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Mar 14 21:26:02 2019 New Revision: 356231 URL: http://llvm.org/viewvc/llvm-project?rev=356231&view=rev Log: Remove an assert in template pack deduction during nested instantiation. Modified: cfe/trunk/lib/Sema/SemaTemplateInstantiateDecl.cpp cfe/trunk/test/SemaTem

r359740 - Consume unexpected "template" keywords after "using"

2019-05-01 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed May 1 16:33:49 2019 New Revision: 359740 URL: http://llvm.org/viewvc/llvm-project?rev=359740&view=rev Log: Consume unexpected "template" keywords after "using" The parser was dealing with unexpected "template" keywords after "using" keywords too late and putting the pars

r359960 - Reduce amount of work ODR hashing does.

2019-05-03 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri May 3 21:22:33 2019 New Revision: 359960 URL: http://llvm.org/viewvc/llvm-project?rev=359960&view=rev Log: Reduce amount of work ODR hashing does. When a FunctionProtoType is in the original type in a DecayedType, the decayed type is a PointerType which points back the o

Re: r359960 - Reduce amount of work ODR hashing does.

2019-05-06 Thread Richard Trieu via cfe-commits
ers back to the outer class type, etc) they > don't cause indefinite cycles? Should that infrastructure have caught > these cases & avoided the redundant work? > > I'm curious to understand better how these things work/overlap/or don't. > > On Fri, May

Re: r359960 - Reduce amount of work ODR hashing does.

2019-05-07 Thread Richard Trieu via cfe-commits
some sort of cycle breaking infrastructure - > >> so that if the same type is seen more than once (eg: classes have > >> members that have pointers back to the outer class type, etc) they > >> don't cause indefinite cycles? Should that infrastructure have caug

r342774 - Make compare function in r342648 have strict weak ordering.

2018-09-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 21 14:20:33 2018 New Revision: 342774 URL: http://llvm.org/viewvc/llvm-project?rev=342774&view=rev Log: Make compare function in r342648 have strict weak ordering. Comparison functions used in sorting algorithms need to have strict weak ordering. Remove the assert an

r342794 - Update smart pointer detection for thread safety analysis.

2018-09-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 21 18:50:52 2018 New Revision: 342794 URL: http://llvm.org/viewvc/llvm-project?rev=342794&view=rev Log: Update smart pointer detection for thread safety analysis. Objects are determined to be smart pointers if they have both a star and arrow operator. Some implementa

r312842 - Catch more cases with -Wenum-compare

2017-09-08 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 8 17:25:05 2017 New Revision: 312842 URL: http://llvm.org/viewvc/llvm-project?rev=312842&view=rev Log: Catch more cases with -Wenum-compare Treat typedef enum as named enums instead of anonymous enums. Anonymous enums are ignored by the warning, so previously, typed

r314035 - Fix unused variable warning in non-debug builds.

2017-09-22 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 22 15:16:13 2017 New Revision: 314035 URL: http://llvm.org/viewvc/llvm-project?rev=314035&view=rev Log: Fix unused variable warning in non-debug builds. Modified: cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp Modified: cfe/trunk/lib/CrossTU/CrossTranslationUnit.

r362187 - Redirect test output to /dev/null

2019-05-30 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu May 30 19:23:33 2019 New Revision: 362187 URL: http://llvm.org/viewvc/llvm-project?rev=362187&view=rev Log: Redirect test output to /dev/null Modified: cfe/trunk/test/Driver/armv8.1m.main.s Modified: cfe/trunk/test/Driver/armv8.1m.main.s URL: http://llvm.org/viewvc/

Re: LLVM Types

2019-08-06 Thread Richard Trieu via cfe-commits
To avoid confusion, there's both a LLVM Type and a Clang Type. The LLVM Type is used in the LLVM IR while the Clang Type represents types in a language, like C++. The Clang Doxygen pages have this hierarchy, although it's truncated because Type is so large, but you can click through to get the mi

r368237 - Update fix-it hints for std::move warnings.

2019-08-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Aug 7 17:12:51 2019 New Revision: 368237 URL: http://llvm.org/viewvc/llvm-project?rev=368237&view=rev Log: Update fix-it hints for std::move warnings. Fix -Wpessimizing-move and -Wredundant-move when warning on initializer lists. The new fix-it hints for removing the std

r368244 - Inline diagnostic text into .td file. NFC.

2019-08-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Aug 7 18:45:31 2019 New Revision: 368244 URL: http://llvm.org/viewvc/llvm-project?rev=368244&view=rev Log: Inline diagnostic text into .td file. NFC. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td cfe/trunk/lib/Sema/SemaExpr.cpp Modified: cfe/tr

[clang-tools-extra] r369316 - Fix typo. "piont" => "point"

2019-08-19 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon Aug 19 17:28:21 2019 New Revision: 369316 URL: http://llvm.org/viewvc/llvm-project?rev=369316&view=rev Log: Fix typo. "piont" => "point" Found by Chris Morris (cwmorris). Modified: clang-tools-extra/trunk/clang-tidy/abseil/DurationConversionCastCheck.cpp clang-t

r364119 - [ODRHash] Skip some typedef types.

2019-06-21 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jun 21 17:32:19 2019 New Revision: 364119 URL: http://llvm.org/viewvc/llvm-project?rev=364119&view=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

r365727 - Increase the number of parser diagnostics.

2019-07-10 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jul 10 19:54:15 2019 New Revision: 365727 URL: http://llvm.org/viewvc/llvm-project?rev=365727&view=rev Log: Increase the number of parser diagnostics. The reserved range for parser diagnostics is getting close to being filled, so increase the space for them. Modified:

r348907 - Move PCHContainerOperations from Frontend to Serialization

2018-12-11 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Dec 11 18:53:59 2018 New Revision: 348907 URL: http://llvm.org/viewvc/llvm-project?rev=348907&view=rev Log: Move PCHContainerOperations from Frontend to Serialization Fix a layering violation. Frontend depends on Serialization, so anything used by both should be in Seria

r349125 - Fix up diagnostics.

2018-12-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Dec 13 19:35:10 2018 New Revision: 349125 URL: http://llvm.org/viewvc/llvm-project?rev=349125&view=rev Log: Fix up diagnostics. Move some diagnostics around between Diagnostic*Kinds.td files. Diagnostics used in multiple places were moved to DiagnosticCommonKinds.td. Di

r349230 - Move static analyzer core diagnostics to common.

2018-12-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 14 18:30:16 2018 New Revision: 349230 URL: http://llvm.org/viewvc/llvm-project?rev=349230&view=rev Log: Move static analyzer core diagnostics to common. Modified: cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td cfe/trunk/include/clang/Basic/DiagnosticFr

r349237 - Fix includes and dependencies for libclang

2018-12-14 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 14 20:25:19 2018 New Revision: 349237 URL: http://llvm.org/viewvc/llvm-project?rev=349237&view=rev Log: Fix includes and dependencies for libclang Remove unneeded includes Add needed include Remove dependency on Serialization Modified: cfe/trunk/tools/libclang/CI

r350143 - Add vtable anchor to classes.

2018-12-28 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Dec 28 18:02:30 2018 New Revision: 350143 URL: http://llvm.org/viewvc/llvm-project?rev=350143&view=rev Log: Add vtable anchor to classes. Modified: cfe/trunk/include/clang/AST/DeclCXX.h cfe/trunk/include/clang/AST/DeclTemplate.h cfe/trunk/include/clang/Lex/Mod

Re: r350143 - Add vtable anchor to classes.

2019-01-04 Thread Richard Trieu via cfe-commits
s in the coding standard - is there any particular > other motivation for this? (Given you've been doing layering cleanup - I'm > wondering fi this is an interesting workaround for some layering problems, > for instance?) > > On Sat, Dec 29, 2018 at 1:05 PM Richard Trie

r350792 - Refactor declarations of ASTContext allocate functions into its own header.

2019-01-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jan 9 19:23:25 2019 New Revision: 350792 URL: http://llvm.org/viewvc/llvm-project?rev=350792&view=rev Log: Refactor declarations of ASTContext allocate functions into its own header. Forward declarations of the allocate functions combine with the forward declaration of t

[clang-tools-extra] r350797 - Remove unnecessary include.

2019-01-09 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Jan 9 20:53:10 2019 New Revision: 350797 URL: http://llvm.org/viewvc/llvm-project?rev=350797&view=rev Log: Remove unnecessary include. QuerySession.h does not need anything from Query.h, so it does not need to include it. Modified: clang-tools-extra/trunk/clang-quer

r350913 - Fix header issues.

2019-01-10 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Thu Jan 10 17:32:35 2019 New Revision: 350913 URL: http://llvm.org/viewvc/llvm-project?rev=350913&view=rev Log: Fix header issues. Several headers would fail to compile if other headers were not previously included. The usual issue is that a class is forward declared, but th

r372448 - Improve -Wtautological-overlap-compare

2019-09-20 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 20 19:37:10 2019 New Revision: 372448 URL: http://llvm.org/viewvc/llvm-project?rev=372448&view=rev Log: Improve -Wtautological-overlap-compare Allow this warning to detect a larger number of constant values, including negative numbers, and handle non-int types better.

r372453 - Merge and improve code that detects same value in comparisons.

2019-09-20 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 20 20:02:26 2019 New Revision: 372453 URL: http://llvm.org/viewvc/llvm-project?rev=372453&view=rev Log: Merge and improve code that detects same value in comparisons. -Wtautological-overlap-compare and self-comparison from -Wtautological-compare relay on detecting the

r372454 - Fix bad APInt compare.

2019-09-20 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Sep 20 21:18:54 2019 New Revision: 372454 URL: http://llvm.org/viewvc/llvm-project?rev=372454&view=rev Log: Fix bad APInt compare. APInt comparison require both to have the same bitwidth. Since only the value is needed, use the compare function APInt::isSameValue instead

r373421 - Revert r368237 - Update fix-it hints for std::move warnings.

2019-10-01 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Oct 1 19:32:15 2019 New Revision: 373421 URL: http://llvm.org/viewvc/llvm-project?rev=373421&view=rev Log: Revert r368237 - Update fix-it hints for std::move warnings. r368237 attempted to improve fix-its for move warnings, but introduced some regressions to -Wpessimizin

r329869 - [ODRHash] Skip more types hashing TypedefType

2018-04-11 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Apr 11 19:26:49 2018 New Revision: 329869 URL: http://llvm.org/viewvc/llvm-project?rev=329869&view=rev Log: [ODRHash] Skip more types hashing TypedefType To get the underlying type for TypedefType's, also skip ElaboratedType's. Modified: cfe/trunk/lib/AST/ODRHash.cpp

r330074 - [ODRHash] Support pointer and reference types.

2018-04-13 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Apr 13 15:34:43 2018 New Revision: 330074 URL: http://llvm.org/viewvc/llvm-project?rev=330074&view=rev Log: [ODRHash] Support pointer and reference types. Recommit r328404 which was reverted in rL328404. r329869 fixed the issue that caused the revert. Modified: cfe/

r330789 - [ODRHash] Hash template arguments of methods.

2018-04-24 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Tue Apr 24 17:31:15 2018 New Revision: 330789 URL: http://llvm.org/viewvc/llvm-project?rev=330789&view=rev Log: [ODRHash] Hash template arguments of methods. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td cfe/trunk/lib/AST/ODRHash.cpp cfe/

r330887 - Switch to Clang's isDigit function.

2018-04-25 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Wed Apr 25 16:50:55 2018 New Revision: 330887 URL: http://llvm.org/viewvc/llvm-project?rev=330887&view=rev Log: Switch to Clang's isDigit function. std::isdigit can be overloaded, causing the template deduction to fail. Use Clang's isDigit function which to avoid this. Swit

r331706 - Move test input file into same directory as test. NFC

2018-05-07 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Mon May 7 17:29:21 2018 New Revision: 331706 URL: http://llvm.org/viewvc/llvm-project?rev=331706&view=rev Log: Move test input file into same directory as test. NFC Added: cfe/trunk/test/Frontend/Inputs/resource_dir_with_cfi_blacklist/ cfe/trunk/test/Frontend/Inputs

r336475 - Check returned type is valid before using it.

2018-07-06 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jul 6 17:17:25 2018 New Revision: 336475 URL: http://llvm.org/viewvc/llvm-project?rev=336475&view=rev Log: Check returned type is valid before using it. Add a .isNull() check to returned QualType. Fixes PR38077 Modified: cfe/trunk/lib/Sema/SemaCXXScopeSpec.cpp

  1   2   3   4   >