Page MenuHomePhabricator
Feed Advanced Search

Yesterday

v.g.vassilev requested review of D92248: Inform the consumer on invalid template instantiations..
Fri, Nov 27, 2:09 PM
Herald added a project to D57018: Fix lvm::is_trivially_copyable portability issues: Restricted Project.
Fri, Nov 27, 1:59 PM · Restricted Project

Sat, Nov 21

v.g.vassilev added inline comments to D91524: Make iteration over the DeclContext::lookup_result safe..
Sat, Nov 21, 3:29 PM
v.g.vassilev updated the diff for D91524: Make iteration over the DeclContext::lookup_result safe..

Invert the diff, fix a typo.

Sat, Nov 21, 3:20 PM
v.g.vassilev updated the diff for D91524: Make iteration over the DeclContext::lookup_result safe..

Take another approach at solving this issue. Instead we are making the lookup_result safe.

Sat, Nov 21, 3:14 PM

Mon, Nov 16

v.g.vassilev committed rG54e655b3f8dc: Reland "Move the test compiler setup in a common place. NFCI" (authored by v.g.vassilev).
Reland "Move the test compiler setup in a common place. NFCI"
Mon, Nov 16, 5:23 AM
v.g.vassilev closed D91061: Move the test compiler setup in a common place. NFCI.
Mon, Nov 16, 5:23 AM · Restricted Project
v.g.vassilev requested review of D91524: Make iteration over the DeclContext::lookup_result safe..
Mon, Nov 16, 2:48 AM

Sat, Nov 14

v.g.vassilev added a comment to D54986: Make CodeGen choose when to emit vtables..

@rsmith, what was the fate of this patch? I thought it landed but cannot find a trace of it in the master...

Sat, Nov 14, 3:16 PM
v.g.vassilev closed D83174: Teach AttachPreviousImpl to inherit MSInheritanceAttr attribute.

Landed in aca191cce1c4dbab28a65cfe4caa6348e698a2b3

Sat, Nov 14, 3:09 PM · Restricted Project
v.g.vassilev abandoned D34912: Handle cases where the value is too large to fit into the underlying type..

Closing this in favor of D77598

Sat, Nov 14, 3:02 PM
v.g.vassilev abandoned D53925: [modules] Defer emission of inline key functions..

Landed as D54986

Sat, Nov 14, 3:00 PM
v.g.vassilev committed rG23cc838099e1: Add forgotten in 888d06d file. (authored by v.g.vassilev).
Add forgotten in 888d06d file.
Sat, Nov 14, 1:33 PM
v.g.vassilev committed rG888d06dfb8b5: Move the test compiler setup in a common place. NFCI (authored by v.g.vassilev).
Move the test compiler setup in a common place. NFCI
Sat, Nov 14, 1:31 PM
v.g.vassilev closed D91061: Move the test compiler setup in a common place. NFCI.
Sat, Nov 14, 1:30 PM · Restricted Project

Tue, Nov 10

v.g.vassilev updated the diff for D91061: Move the test compiler setup in a common place. NFCI.

Add forgotten file.

Tue, Nov 10, 11:52 AM · Restricted Project

Mon, Nov 9

v.g.vassilev requested review of D91061: Move the test compiler setup in a common place. NFCI.
Mon, Nov 9, 2:33 AM · Restricted Project

Sat, Nov 7

v.g.vassilev requested review of D90998: Allow in-place compound statment body replacement.
Sat, Nov 7, 12:52 AM

Oct 27 2020

v.g.vassilev added a reviewer for D89749: SourceManager: Don't allocate an SLocEntry until it's loaded: rsmith.

Unfortunately, the patch does not apply against llvm9 (which is what we have as experimental setup) not to speak against llvm5 which is our production setup.

Oct 27 2020, 9:44 AM

Oct 26 2020

v.g.vassilev added a comment to D89749: SourceManager: Don't allocate an SLocEntry until it's loaded.

Thanks for the patch!! This is a super hot place for us (mostly due to boost). I will try it on our end and let you know!

Oct 26 2020, 8:26 AM

Oct 8 2020

v.g.vassilev added a comment to D88780: Allow interfaces to operate on in-memory buffers with no source location info..

@shafik, I suppose that with a good amount of effort we may be able to test it in a unittest setup.

Oct 8 2020, 12:43 PM · Restricted Project

Sep 4 2020

v.g.vassilev committed rG2c9dbcda4f71: [modules] Correctly parse LateParsedTemplates in case of dependent modules. (authored by gargvaibhav64).
[modules] Correctly parse LateParsedTemplates in case of dependent modules.
Sep 4 2020, 4:40 AM
v.g.vassilev closed D86514: Correctly parse LateParsedTemplates in case of multiple dependent modules.
Sep 4 2020, 4:39 AM · Restricted Project

Jul 16 2020

v.g.vassilev removed a reviewer for D83174: Teach AttachPreviousImpl to inherit MSInheritanceAttr attribute: aaron.ballman.

Also, I would like to add that the current test present in this diff does not fail in the existing system but it was the best I and Vassil could come up with to replicate our problem.

Is there a way to run something like creduce to come up with a test case that does replicate the issue? I'd like to ensure we have a test case that fails without the patch applied so that we can be sure we don't regress the behavior.

Jul 16 2020, 11:11 AM · Restricted Project

Jun 22 2020

v.g.vassilev committed rG46ea465b5b74: Return false if the identifier is not in the global module index. (authored by v.g.vassilev).
Return false if the identifier is not in the global module index.
Jun 22 2020, 1:34 AM
v.g.vassilev closed D81077: If the identifier is unknown, return false..
Jun 22 2020, 1:34 AM · Restricted Project
v.g.vassilev accepted D82169: Don't install clang-import-test.

Makes sense to me!

Jun 22 2020, 1:01 AM · Restricted Project

Jun 4 2020

v.g.vassilev updated subscribers of D81077: If the identifier is unknown, return false..
Jun 4 2020, 4:18 AM · Restricted Project

Jun 3 2020

v.g.vassilev created D81077: If the identifier is unknown, return false..
Jun 3 2020, 3:48 AM · Restricted Project

May 2 2020

v.g.vassilev added a comment to D79290: Update suffix check and cast non-suffix types.

@hubert.reinterpretcast, this patch is part of our internal forks which we would like to put upstream. The author of the previous patch does not have bandwidth to work on it and @reikdas kindly volunteered to take over. I can close the previous one if that's preferable.

May 2 2020, 10:47 PM · Restricted Project

Apr 7 2020

v.g.vassilev added reviewers for D77598: Integral template argument suffix and cast printing: rsmith, bruno.
Apr 7 2020, 9:12 AM · Restricted Project

Nov 28 2019

v.g.vassilev added a comment to D69309: Support template instantiation in the expression evaluator.

https://reviews.llvm.org/D41416 could be relevant. I am not an expert but I think when reading the DWARF you could only register 'lazy' specializations which will be imported only when really required.

Nov 28 2019, 10:36 PM · Restricted Project

Sep 26 2019

Herald added a project to D36368: Fix type printing of array template args: Restricted Project.

ping...

Sep 26 2019, 10:05 PM · Restricted Project, Restricted Project

Jul 16 2019

v.g.vassilev added a comment to D42026: Allow the llvm bindings to be disabled.

Thanks!

Jul 16 2019, 6:23 AM · Restricted Project

Jul 4 2019

v.g.vassilev committed rG16de81c15850: [modules] Add PP callbacks for entering and leaving a submodule. (authored by v.g.vassilev).
[modules] Add PP callbacks for entering and leaving a submodule.
Jul 4 2019, 12:08 PM
v.g.vassilev closed D62293: [modules] Add PP callbacks for entering and leaving a submodule..

r365153.

Jul 4 2019, 12:08 PM · Restricted Project

Jun 27 2019

v.g.vassilev updated the diff for D62293: [modules] Add PP callbacks for entering and leaving a submodule..

Add comments, add missing callbacks. Thanks Richard!

Jun 27 2019, 12:59 PM · Restricted Project

May 23 2019

v.g.vassilev created D62293: [modules] Add PP callbacks for entering and leaving a submodule..
May 23 2019, 2:08 AM · Restricted Project

Mar 25 2019

v.g.vassilev added a comment to D16761: clang-cl: Support loading plugins on Windows.

Indeed. IIUC, here is no way clang-cl to load plugins if compiled BUILD_SHARED_LIB=0. It is not clear to me if that's a design limitation or something else. I'd love to be able to load plugins on a 'standard' clang-cl instance.

Mar 25 2019, 5:23 AM

Mar 24 2019

v.g.vassilev updated subscribers of D16761: clang-cl: Support loading plugins on Windows.

@chapuni, you have reverted this patch and the commit message says "investigating". Did you come to a conclusion? Can we reland it?

Mar 24 2019, 1:46 PM

Nov 25 2018

v.g.vassilev created D54880: Ignore gcc's stack-clash-protection flag.
Nov 25 2018, 1:03 PM · Restricted Project

Nov 20 2018

v.g.vassilev added a comment to D35559: [CMake][Modules] Tweak Modules-unfriendly builds.

Is this change still relevant?

Nov 20 2018, 2:01 AM
v.g.vassilev added a comment to D43871: [modules] No longer include stdlib.h from mm_malloc.h..

ping...

Nov 20 2018, 1:59 AM

Nov 17 2018

v.g.vassilev added a comment to D54097: Save memory in ASTReader by using DenseMap instead of vector.

Ping...

Nov 17 2018, 1:13 PM

Oct 31 2018

v.g.vassilev added a reviewer for D53925: [modules] Defer emission of inline key functions.: rjmccall.
Oct 31 2018, 3:48 AM
v.g.vassilev added a comment to D53925: [modules] Defer emission of inline key functions..

@rsmith, @bruno could you verify this patch on your codebases?

Oct 31 2018, 3:47 AM
v.g.vassilev created D53925: [modules] Defer emission of inline key functions..
Oct 31 2018, 3:47 AM

Aug 27 2018

v.g.vassilev updated the diff for D51295: Allow resetting of NumCreatedFIDsForFileID.

Add parens.

Aug 27 2018, 5:35 AM
v.g.vassilev created D51295: Allow resetting of NumCreatedFIDsForFileID.
Aug 27 2018, 4:48 AM

Jun 28 2018

v.g.vassilev added a comment to D47118: [modules] Print input files when -module-file-info file switch is passed..

It looks like the diagnostic options and few others were moved out from the control block in r297655 by Manman Ren. The header files are still part of the control block. The best I can do is buffer that information and print it out last. Is that what we want?

Jun 28 2018, 8:30 AM
v.g.vassilev retitled D48524: [ODRHash] Do not rely on Type* when computing the hash. from [ODRHash] Rip out the registration of Type* in TypeMap to [ODRHash] Do not rely on Type* when computing the hash..
Jun 28 2018, 12:51 AM

Jun 27 2018

v.g.vassilev added a comment to D48524: [ODRHash] Do not rely on Type* when computing the hash..

@rtrieu, your comment was along the lines of what I was thinking of. I have updated the patch. Without testing it on large scale, I the current change seems to work for us.

Jun 27 2018, 7:30 AM
v.g.vassilev updated the diff for D48524: [ODRHash] Do not rely on Type* when computing the hash..

Remove the TypeMap

Jun 27 2018, 7:29 AM

Jun 23 2018

v.g.vassilev created D48524: [ODRHash] Do not rely on Type* when computing the hash..
Jun 23 2018, 2:26 PM

May 22 2018

v.g.vassilev accepted D47247: Fix unaligned memory access when reading INPUT_FILE_OFFSETS data.

LGTM!

May 22 2018, 11:55 PM

May 20 2018

v.g.vassilev created D47118: [modules] Print input files when -module-file-info file switch is passed..
May 20 2018, 8:04 AM
v.g.vassilev added a comment to D42026: Allow the llvm bindings to be disabled.

r332816.

May 20 2018, 1:43 AM · Restricted Project

Apr 19 2018

v.g.vassilev added reviewers for D37475: Make MultiplexASTDeserializationListener part of the API [NFC]: v.g.vassilev, rsmith, bruno.
Apr 19 2018, 1:56 AM

Apr 11 2018

v.g.vassilev added inline comments to D43871: [modules] No longer include stdlib.h from mm_malloc.h..
Apr 11 2018, 12:35 AM

Mar 24 2018

v.g.vassilev abandoned D43696: Reduce hash collisions for reference and pointer types.

Ok, that's great! Sorry for the delay and thanks for landing a similar patch. Btw, we should probably find a more terse way to test the ODRHash, eg. with unittests.

Mar 24 2018, 2:55 PM

Mar 14 2018

v.g.vassilev added inline comments to D44435: CUDA ctor/dtor Module-Unique Symbol Name.
Mar 14 2018, 2:35 PM

Mar 10 2018

v.g.vassilev added a comment to D40731: Integrate CHash into CLang.

I am not sure if this is the right time but maybe testing and comparing the behavior of this patch against something such as the ODRHash will help you, me (and probably others) to understand the current patch better.

Mar 10 2018, 2:41 AM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Add sanity check.

Mar 10 2018, 2:31 AM

Feb 24 2018

v.g.vassilev added inline comments to D43696: Reduce hash collisions for reference and pointer types.
Feb 24 2018, 2:36 AM

Feb 23 2018

v.g.vassilev created D43696: Reduce hash collisions for reference and pointer types.
Feb 23 2018, 2:15 PM
v.g.vassilev added a comment to D42026: Allow the llvm bindings to be disabled.

Reading back the patch, we are turning on the bindings by default ;)

Feb 23 2018, 2:02 PM · Restricted Project
v.g.vassilev added a comment to D37797: Fix crash in Sema when wrongly assuming VarDecl init is not value dependent..

Could you double-check if we can add this to an existing test? Would test/SemaCXX/pr25181-crash-on-invalid.cpp fit and potentially rename it.

Feb 23 2018, 2:00 PM
v.g.vassilev updated subscribers of D32111: [modules] Attempt to fix PR31905 - #include "stddef.h" breaks module map search paths; causes redefinitions..
Feb 23 2018, 1:52 PM

Feb 21 2018

v.g.vassilev added a comment to D42026: Allow the llvm bindings to be disabled.

ping...

Feb 21 2018, 2:17 AM · Restricted Project
v.g.vassilev added a comment to D34464: lit: Make sure testnames are unicode strings.

ping2...

Feb 21 2018, 2:02 AM

Jan 13 2018

v.g.vassilev created D42026: Allow the llvm bindings to be disabled.
Jan 13 2018, 2:39 AM · Restricted Project

Jan 12 2018

v.g.vassilev accepted D42001: [Driver] Add "did you mean?" suggestions to -cc1as.

LGTM!

Jan 12 2018, 10:53 AM

Jan 10 2018

v.g.vassilev accepted D41912: [Driver] Test for correct '--version' suggestion.

LGMT! We can rely on post-commit reviews for changes like this one.

Jan 10 2018, 11:36 AM

Jan 9 2018

v.g.vassilev accepted D41867: [Frontend] Remove unused FileMgr in pp arg parse.

LGTM!

Jan 9 2018, 12:21 PM

Jan 5 2018

v.g.vassilev added a comment to D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Finer grained clang stats can be seen here.

Jan 5 2018, 2:49 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Reduce further the deserializations from 451 to 449 by providing a more complete implementation of ODRHash::AddTemplateArgument.

Jan 5 2018, 2:32 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Reduce hash collisions for template specializations.

Jan 5 2018, 2:27 PM
v.g.vassilev created D41785: Print the qualified name when dumping deserialized decls..
Jan 5 2018, 2:03 PM
v.g.vassilev added inline comments to D41733: [Driver] Suggest correctly spelled driver options.
Jan 5 2018, 4:46 AM
v.g.vassilev added a comment to D41732: [Option] Add 'findNearest' method to catch typos.

+1 Thanks for working on this. Great to see this happening!

Jan 5 2018, 4:43 AM

Jan 4 2018

v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Reverse template specialization order.

Jan 4 2018, 2:58 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Zero IsPartial and improve comments in assert and free text.

Jan 4 2018, 2:44 PM
v.g.vassilev added a comment to D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

I have created a simple (-ish) benchmark targeted to stress test modules with a good number of template specializations. I use the mp11 library from boost. I am building two modules which contain around 1K specializations (mostly full specializations). The example can be found here. I use a small fraction of them. This patch deserializes (only) 1117 specializations for around 0.2 seconds whereas without it clang deserializes 1905 specializations for 0.4 s.

Jan 4 2018, 2:42 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Loading of lazy partial template specializations should not trigger loading of full template specializations.

Jan 4 2018, 1:21 PM

Jan 3 2018

v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Address comments:

  • Fix style;
  • Do not potentially deserialize a specialization in debug mode.
Jan 3 2018, 3:18 PM

Jan 2 2018

v.g.vassilev added inline comments to D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..
Jan 2 2018, 1:55 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Teach ASTReader::CompleteRedeclChain to load only the template specializations with the same template argument list.

Jan 2 2018, 1:54 PM
v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Address inline comments: order the read operations.

Jan 2 2018, 1:46 PM

Dec 19 2017

v.g.vassilev updated the diff for D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..

Fixed a comment typo.

Dec 19 2017, 2:58 PM
v.g.vassilev created D41416: [modules] [pch] Do not deserialize all lazy template specializations when looking for one..
Dec 19 2017, 2:55 PM

Dec 14 2017

v.g.vassilev closed D40901: Refactor lazy loading of template specializations. NFC.

Thanks! Landed in r320763.

Dec 14 2017, 3:31 PM

Dec 6 2017

v.g.vassilev updated the diff for D40901: Refactor lazy loading of template specializations. NFC.

Fix preexisting comment typo.

Dec 6 2017, 8:41 AM
v.g.vassilev created D40901: Refactor lazy loading of template specializations. NFC.
Dec 6 2017, 8:17 AM

Nov 21 2017

v.g.vassilev added a comment to D29951: Load lazily the template specialization in multi-module setups..

For the record: relanded in r306903.

Nov 21 2017, 2:20 PM

Oct 31 2017

v.g.vassilev accepted D39416: [modules] Correctly overload getModule in the MultiplexExternalSemaSource.

LGTM!

Oct 31 2017, 7:37 AM

Oct 27 2017

v.g.vassilev added inline comments to D39342: [Bash-autocompletion] Pass all flags in shell command-line to Clang.
Oct 27 2017, 12:30 PM

Oct 20 2017

v.g.vassilev added a comment to D39002: [cmake] Use find_package to discover zlib.

Reverted in r316153...

Oct 20 2017, 8:29 AM

Oct 19 2017

v.g.vassilev added a comment to D39002: [cmake] Use find_package to discover zlib.

Revised patch landed in r316150.

Oct 19 2017, 1:05 AM

Oct 17 2017

v.g.vassilev added a comment to D39002: [cmake] Use find_package to discover zlib.

I assume this would fix it:

Oct 17 2017, 2:11 PM
v.g.vassilev added a comment to D39002: [cmake] Use find_package to discover zlib.

Reverted in r316029 because the bots got angry at me:

Oct 17 2017, 2:02 PM