Page MenuHomePhabricator

dexonsmith (Duncan P. N. Exon Smith)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 18 2014, 10:33 AM (278 w, 5 d)

Recent Activity

Thu, Jul 18

dexonsmith accepted D64945: [clang-scan-deps] Dependency directives source minimizer: handle #pragma once.

LGTM after you add the testcase I mention.

Thu, Jul 18, 1:46 PM · Restricted Project, Restricted Project

Fri, Jul 12

dexonsmith added inline comments to D64600: [ObjC] Add an attribute that "clamps" signed char BOOLs to {0,1}.
Fri, Jul 12, 3:18 PM · Restricted Project

Wed, Jul 10

dexonsmith accepted D64525: [clang-scan-deps] Dependency directives source minimizer: single quotes are not digit separators after a valid character literal prefix.

LGTM, but I have a suggestion inline for another approach.

Wed, Jul 10, 11:49 AM · Restricted Project, Restricted Project

Mon, Jul 8

dexonsmith added a reviewer for D64191: [libcxxabi] Don't process exceptions in cxa_handlers when they're disabled: Bigcheese.
Mon, Jul 8, 2:20 PM · Restricted Project

Wed, Jul 3

dexonsmith added a comment to D64149: [clang-scan-deps] use `-Wno-error` when scanning for dependencies.

Is warning output suppressed? If so, should we just/also disable all warnings? (IIRC, the flag is -w.)

Wed, Jul 3, 3:25 PM · Restricted Project, Restricted Project

Wed, Jun 26

dexonsmith added a comment to D63048: Update __VERSION__ to remove the hardcoded 4.2.1 version.
Wed, Jun 26, 2:31 PM · Restricted Project, Restricted Project

Tue, Jun 25

dexonsmith updated subscribers of D60480: [libc++] Integrate the PSTL into libc++.
Tue, Jun 25, 1:44 PM · Restricted Project, Restricted Project

Jun 19 2019

dexonsmith added a comment to rL336869: IR: Skip -print-*-all after -print-*.

Hello, can you please explain a bit more about this change? What kind of redundant outputs avoided by this? Thanks!

Jun 19 2019, 9:03 AM

Jun 14 2019

dexonsmith added a comment to D62742: [OpaquePtr] BitcodeReader: don't rely on Types derived from a Value to provide pointer structure.

I don't fully have the direction you've in mind/that this is a part of in my head - but roughly enough to give a thumbs-up here :)

Jun 14 2019, 4:02 PM · Restricted Project

May 31 2019

dexonsmith added a comment to D62428: [libcxx] Slightly improved policy for handling experimental features.
In D62428#1519664, @jfb wrote:

The deletions are fine, the doc updates are good, but I think deprecation warnings are not.
Things that are going to happen in the future (a year from now, say) or never (if people don't update their tools) don't belong in the warning spew for every single build

May 31 2019, 10:28 AM · Restricted Project, Restricted Project

May 30 2019

dexonsmith added a comment to D60713: [IR] Add DISuprogram and DIE for func decl of an external.

Prior to this patch Function declarations may not have a !dbg attachment, because there was no use-case for it. With call site debug info, I think it would make sense to revisit this rule. When I made my previous comment, I thought that the updated rule could be:

(1) function definitions may have a distinct DISubprogram attached (as before)
(2) function declarations may have a unique DISubprogram attached (new)

If I read the code in Verifier.cpp correctly, then it is legal for a unique DISubprogram to not have a unit, which would cause the declaration-DISubprograms to be uniqued during LTO.

so the (2) should probably be:

(2) function declarations may have a unique DISubprogram attached and it may not have a unit: field

I've CC'ed Duncan in case he can catch a logic error with this reasoning.

May 30 2019, 9:01 AM · Restricted Project, debug-info

May 29 2019

dexonsmith added a comment to D62618: [libcxx] Make std::tuple<> trivially constructible.

Changing the triviality of a type can be ABI breaking because it will change the calling conventions the compiler uses.
We have hacks in pair to keep it non-trivial on some platforms for this reason.

May 29 2019, 1:33 PM · Restricted Project, Restricted Project
dexonsmith added inline comments to D62319: IR: add 'byval(<ty>)' variant to 'byval' function parameters.
May 29 2019, 10:08 AM · Restricted Project

May 27 2019

dexonsmith added a comment to D62493: [Driver] Always use Unix-style paths in the Darwin driver.

This doesn’t look okay to me, because this would prevent building for Darwin when running on Windows. I added a couple of reviewers that have Windows experience and might have ideas for how to fix the tests without breaking the portability of the driver.

May 27 2019, 8:35 PM · Restricted Project
dexonsmith added reviewers for D62493: [Driver] Always use Unix-style paths in the Darwin driver: Bigcheese, rnk.
May 27 2019, 8:33 PM · Restricted Project

May 25 2019

dexonsmith added a parent revision for D62458: Frontend: Add CompilerInvocation and DiagnosticsEngine to the builder: D62457: Frontend: Create basic CompilerInstanceBuilder.
May 25 2019, 3:45 PM
dexonsmith added a child revision for D62457: Frontend: Create basic CompilerInstanceBuilder: D62458: Frontend: Add CompilerInvocation and DiagnosticsEngine to the builder.
May 25 2019, 3:45 PM
dexonsmith created D62457: Frontend: Create basic CompilerInstanceBuilder.
May 25 2019, 3:45 PM
dexonsmith created D62458: Frontend: Add CompilerInvocation and DiagnosticsEngine to the builder.
May 25 2019, 3:45 PM
dexonsmith committed rC361708: Add missing newline at end of file.
Add missing newline at end of file
May 25 2019, 3:39 PM
dexonsmith committed rGd4a9cae96500: Add missing newline at end of file (authored by dexonsmith).
Add missing newline at end of file
May 25 2019, 3:39 PM
dexonsmith committed rL361708: Add missing newline at end of file.
Add missing newline at end of file
May 25 2019, 3:35 PM

May 23 2019

dexonsmith added a comment to D62228: Make `vector` unconditionally move elements when exceptions are disabled..

There was a discussion on the mailing list that (to my mind) petered out with no particular conclusion as to whether or not this was a good idea.

Louis and I talked more offline and there was some consensus that this minimal change was more acceptable.
But if there are additional concerns they can be raised here.

May 23 2019, 9:38 AM · Restricted Project

May 14 2019

dexonsmith accepted D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format.

LGTM too.

May 14 2019, 1:59 PM · Restricted Project

May 9 2019

dexonsmith added a comment to D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format.
In D61547#1495339, @rnk wrote:

Presumably test/Bitcode/upgrade-global-ctors.ll.bc should remain unchanged, i.e. not be regenerated? We want it to use the pre-upgraded, two field form.

I used llvm-as without this patch to generate it. I added anither test to check the upgrader correctly adds i8* null

What's the problem with using the pre-existing file?

test/Bitcode/upgrade-global-ctors.ll.bc uses zeroinitializer, it doesn't check if AuoUpgrader adds i8* null as the third field. So I improved the test a bit and made it clear both globals_ctors and global_dtors are upgraded.

May 9 2019, 6:04 PM · Restricted Project
dexonsmith added a comment to D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format.
In D61547#1495339, @rnk wrote:

Presumably test/Bitcode/upgrade-global-ctors.ll.bc should remain unchanged, i.e. not be regenerated? We want it to use the pre-upgraded, two field form.

I used llvm-as without this patch to generate it. I added anither test to check the upgrader correctly adds i8* null

May 9 2019, 11:54 AM · Restricted Project
dexonsmith updated subscribers of D53072: [clang-format] Add a new extra command line option for ide-specific formatting.
May 9 2019, 11:25 AM

May 8 2019

dexonsmith added a comment to D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format.
In D61547#1495339, @rnk wrote:

Presumably test/Bitcode/upgrade-global-ctors.ll.bc should remain unchanged, i.e. not be regenerated? We want it to use the pre-upgraded, two field form.

May 8 2019, 10:57 AM · Restricted Project

May 7 2019

dexonsmith added a reviewer for D58497: Clear the KnownModules cache if the preprocessor is going away: bruno.

+bruno

May 7 2019, 6:00 PM · Restricted Project
dexonsmith requested changes to D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format.

Hmm; it doesn't seem safe to obsolete the 2-field form if we don't have a working bitcode upgrade from 2-field to 3-field.

May 7 2019, 10:42 AM · Restricted Project

May 6 2019

dexonsmith added a comment to D61165: Fix a crash where a [[no_destroy]] destructor was not emitted in an array.

Duncan pointed out eel.is/c++draft/class.init#class.base.init-13, which appears to claim that initialization ends with the execution of the constructor, excluding temporary destructors.

(13) In a non-delegating constructor, initialization proceeds in the following order:
/* list of steps... */
(13.4) Finally, the compound-statement of the constructor body is executed.

John: do you agree with this analysis?

That's talking about constructor bodies.

May 6 2019, 5:37 PM · Restricted Project, Restricted Project
dexonsmith updated subscribers of D60285: Make calls into the pthread library use weak symbols..
May 6 2019, 4:47 PM · Restricted Project
dexonsmith added a reviewer for D61547: [IR] Disallow llvm.global_ctors and llvm.global_dtors of the 2-field form in textual format: steven_wu.

The 3-field form was introduced by D3499 in 2014 and the legacy 2-field
form was planned to be removed in LLVM 4.0

May 6 2019, 11:31 AM · Restricted Project
dexonsmith edited reviewers for D61369: [DebugInfo] Delete TypedDINodeRef, added: rnk; removed: dexonsmith.

+rnk

May 6 2019, 9:26 AM · Restricted Project

Apr 30 2019

dexonsmith added a reviewer for D61346: [CMake] Do not use libtool on Apple platforms when building LLVM with (full) LTO.: steven_wu.

Shouldn't this just send DYLD_LIBRARY_PATH through to the second stage, so that ar and ranlib use the just-built libLTO.dylib?

Apr 30 2019, 3:50 PM · Restricted Project

Apr 29 2019

dexonsmith accepted D60924: [ThinLTO] Adding architecture name into saved object filename.

LGTM.

Apr 29 2019, 2:29 PM · Restricted Project

Apr 19 2019

dexonsmith added a comment to D60924: [ThinLTO] Adding architecture name into saved object filename.

Is there a good way to test this?

Not currently in LLVM. It might be possible to add a test in clang repo which requires DARWIN and ld64.

Maybe we can split out the function to generate the filename, and add a unit test for that?

That almost looks like a test for path::append. Not sure if there is much value to add that.

Apr 19 2019, 4:38 PM · Restricted Project
dexonsmith added a comment to D60924: [ThinLTO] Adding architecture name into saved object filename.

Is there a good way to test this?

Not currently in LLVM. It might be possible to add a test in clang repo which requires DARWIN and ld64.

Apr 19 2019, 2:56 PM · Restricted Project
dexonsmith added a comment to D60924: [ThinLTO] Adding architecture name into saved object filename.

Is there a good way to test this?

Apr 19 2019, 2:40 PM · Restricted Project
dexonsmith added a comment to D60748: Fix i386 struct and union parameter alignment.

I suspect Darwin also doesn't want to take this. We care very little about 32-bit Intel, and part of caring very little is not wanting to spend any effort dealing with the ramifications of ABI breaks. That would apply both to the rule in general and to the vector rule specifically. @dexonsmith, agreed?

Apr 19 2019, 11:18 AM · Restricted Project

Apr 18 2019

dexonsmith added a comment to D60162: [ThinLTO] Add module flags for TargetLibraryInfoImpl and use in LTO backends.

I wonder if we could add a module flag for the TLI, and then store that in the summary. Would it work for both implementations in that case?

Apr 18 2019, 11:28 AM · Restricted Project, Restricted Project
dexonsmith updated subscribers of D60880: [libc++] Use COPYFILE_CLONE from the macOS copyfile(3) API to support APFS clones.

+aprantl

Apr 18 2019, 11:10 AM · Restricted Project

Apr 15 2019

dexonsmith added a comment to D60233: [clang-scan-deps] initial outline of the tool that runs preprocessor to find dependencies over a JSON compilation database.

A comment by whisperity in the origin WIP did not seem to be addressed.
Have you looked at IWYU https://github.com/include-what-you-use/include-what-you-use ?
The end goals of clang-scan-deps and iwyu seem similar so their implementation problems would also be similar.
The iwyu problems doc is https://github.com/include-what-you-use/include-what-you-use/blob/master/docs/WhyIWYUIsDifficult.md

Apr 15 2019, 10:49 AM · Restricted Project, Restricted Project

Apr 2 2019

dexonsmith added a comment to D55463: Introduce a source minimizer that reduces source to directives that might affect the dependency list for a compilation.

Are you planning to do this recursively?
The minimizer does not help much for the following example, while Sema.h contains 10,000 lines of useless code.

#include "clang/Sema/Sema.h"

int foo() {}
Apr 2 2019, 9:38 AM · Restricted Project, Restricted Project

Mar 28 2019

dexonsmith added inline comments to D59930: Parallel STL version updated.
Mar 28 2019, 5:46 PM

Mar 26 2019

dexonsmith added a comment to D59767: [pstl] Indent preprocessor directives as part of the clang-format rules.

I'm in favour of this.

Mar 26 2019, 7:42 PM · Restricted Project
dexonsmith committed rGdb8a74220674: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC (authored by dexonsmith).
Basic: Return a reference from FileManager::getVirtualFileSystem, NFC
Mar 26 2019, 3:31 PM
dexonsmith committed rC357038: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.
Basic: Return a reference from FileManager::getVirtualFileSystem, NFC
Mar 26 2019, 3:31 PM
dexonsmith committed rCTE357038: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.
Basic: Return a reference from FileManager::getVirtualFileSystem, NFC
Mar 26 2019, 3:31 PM
dexonsmith committed rL357038: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.
Basic: Return a reference from FileManager::getVirtualFileSystem, NFC
Mar 26 2019, 3:31 PM
dexonsmith closed D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.

Committed r357038.

Mar 26 2019, 3:30 PM
dexonsmith committed rG1da7eac87c15: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC (authored by dexonsmith).
Frontend: Remove CompilerInstance::VirtualFileSystem, NFC
Mar 26 2019, 3:19 PM
dexonsmith committed rLLDB357037: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Frontend: Remove CompilerInstance::VirtualFileSystem, NFC
Mar 26 2019, 3:19 PM
dexonsmith committed rCTE357037: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Frontend: Remove CompilerInstance::VirtualFileSystem, NFC
Mar 26 2019, 3:18 PM
dexonsmith committed rC357037: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Frontend: Remove CompilerInstance::VirtualFileSystem, NFC
Mar 26 2019, 3:18 PM
dexonsmith committed rL357037: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Frontend: Remove CompilerInstance::VirtualFileSystem, NFC
Mar 26 2019, 3:18 PM
dexonsmith closed D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.

Committed r357037.

Mar 26 2019, 3:18 PM

Mar 20 2019

dexonsmith edited reviewers for D59628: Add support for __attribute__((objc_class_stub)), added: erik.pilkington, arphaman; removed: dexonsmith.

+Erik and Alex, can you take a look at this?

Mar 20 2019, 7:51 PM · Restricted Project
dexonsmith accepted D59489: [libc++][CMake] Clean up some of the libc++ re-exporting logic.

LGTM.

Mar 20 2019, 10:42 AM

Mar 18 2019

dexonsmith accepted D58548: IR: Support parsing numeric block ids, and emit them in textual output..

LGTM!

Mar 18 2019, 3:25 PM · Restricted Project, Restricted Project
dexonsmith updated the diff for D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.

Rebase.

Mar 18 2019, 9:03 AM
dexonsmith added a comment to D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.

... since I noticed that FileManager ...

This kind of implies that we should move the comment from FileManager constructor implementation to the header thus making it an explicit part of the interface contract.

// If the caller doesn't provide a virtual file system, just grab the real
// file system.

Maybe https://reviews.llvm.org/D59388 would be the right place to do it.

Mar 18 2019, 9:00 AM
dexonsmith updated the diff for D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.

Document the default VFS used by the FileManager.

Mar 18 2019, 9:00 AM
dexonsmith added a comment to D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.

Hi Duncan, thanks for working on better interfaces in clang!

I am just wondering - is it safe to have the lifetime of a single object on heap managed by two different IntrusiveRefCntPtr instances?

Mar 18 2019, 8:50 AM

Mar 14 2019

dexonsmith added a parent revision for D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC: D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Mar 14 2019, 2:49 PM
dexonsmith added a child revision for D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC: D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.
Mar 14 2019, 2:49 PM
dexonsmith created D59388: Basic: Return a reference from FileManager::getVirtualFileSystem, NFC.
Mar 14 2019, 2:49 PM
dexonsmith updated the diff for D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.

I slightly simplified the code for CompilerInstance::createFileManager (replaced with an assert), since I noticed that FileManager always constructs its own VFS if it isn't passed one.

Mar 14 2019, 2:25 PM
dexonsmith added reviewers for D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC: Bigcheese, arphaman, bruno.
Mar 14 2019, 1:07 PM
dexonsmith created D59377: Frontend: Remove CompilerInstance::VirtualFileSystem, NFC.
Mar 14 2019, 11:30 AM

Mar 13 2019

dexonsmith added a comment to D58514: Avoid needlessly copying blocks that initialize or are assigned to local auto variables to the heap.

Sorry, I misread the code. If you change the parameter type of bar to BlockTy, the code crashes. If the type is id, it doesn't crash because IRGen copies the block to the heap in foo before passing it to bar.

Mar 13 2019, 5:19 PM · Restricted Project
dexonsmith added a comment to D58514: Avoid needlessly copying blocks that initialize or are assigned to local auto variables to the heap.

That code doesn't crash because the block is retained at the entry of bar and ARC optimizer doesn't remove the retain/release pairs in bar.

Mar 13 2019, 4:33 PM · Restricted Project
dexonsmith added a comment to D58514: Avoid needlessly copying blocks that initialize or are assigned to local auto variables to the heap.

Seems like the chromium code is valid and shouldn't crash. John/Erik what do you think? The following code also crashes with this patch applied.

typedef void (^BlockTy)();

BlockTy sb;
__weak BlockTy wb;

void foo(id a) {
  auto b = ^{ NSLog(@"foo %@", a); };
  wb = b; // block isn't copied to the heap.
  sb = b; // block is copied to the heap.
}

int main() {
  auto x = [NSObject new];
  foo(x);
  sb();
  wb();
  return 0;
}
Mar 13 2019, 4:21 PM · Restricted Project

Mar 12 2019

dexonsmith committed rG70d759b4eb95: Modules: Add LangOptions::CacheGeneratedPCH (authored by dexonsmith).
Modules: Add LangOptions::CacheGeneratedPCH
Mar 12 2019, 11:39 AM
dexonsmith committed rL355950: Modules: Add LangOptions::CacheGeneratedPCH.
Modules: Add LangOptions::CacheGeneratedPCH
Mar 12 2019, 11:39 AM
dexonsmith committed rC355950: Modules: Add LangOptions::CacheGeneratedPCH.
Modules: Add LangOptions::CacheGeneratedPCH
Mar 12 2019, 11:38 AM
dexonsmith closed D59176: Modules: Add LangOptions::CacheGeneratedPCH.

Committed in r355950. Thanks for the review.

Mar 12 2019, 11:38 AM

Mar 11 2019

dexonsmith added inline comments to D59176: Modules: Add LangOptions::CacheGeneratedPCH.
Mar 11 2019, 8:35 PM
dexonsmith updated the diff for D59176: Modules: Add LangOptions::CacheGeneratedPCH.

Updated the constructor call to PCHGenerator in GenerateModuleAction::CreateASTConsumer to use BuildingImplicitModule on its own. Checking where it's set (only in compileModuleImpl), it's exactly the condition we want here.

Mar 11 2019, 8:25 PM
dexonsmith added a comment to D59176: Modules: Add LangOptions::CacheGeneratedPCH.

This commit by itself doesn't change any behavior, right?

Mar 11 2019, 1:56 PM

Mar 9 2019

dexonsmith committed rGb7db2e9f8242: Stop relying on allocator behaviour in modules unit test (authored by dexonsmith).
Stop relying on allocator behaviour in modules unit test
Mar 9 2019, 12:15 PM
dexonsmith committed rC355780: Stop relying on allocator behaviour in modules unit test.
Stop relying on allocator behaviour in modules unit test
Mar 9 2019, 12:14 PM
dexonsmith committed rL355780: Stop relying on allocator behaviour in modules unit test.
Stop relying on allocator behaviour in modules unit test
Mar 9 2019, 12:14 PM
dexonsmith committed rG2fd0d227f6bc: Fix slashes in path references in -Rmodule-import test from r355778 (authored by dexonsmith).
Fix slashes in path references in -Rmodule-import test from r355778
Mar 9 2019, 11:33 AM
dexonsmith committed rC355779: Fix slashes in path references in -Rmodule-import test from r355778.
Fix slashes in path references in -Rmodule-import test from r355778
Mar 9 2019, 11:32 AM
dexonsmith committed rL355779: Fix slashes in path references in -Rmodule-import test from r355778.
Fix slashes in path references in -Rmodule-import test from r355778
Mar 9 2019, 11:32 AM
dexonsmith closed D58893: Modules: Invalidate out-of-date PCMs as they're discovered.

Committed in r355778.

Mar 9 2019, 9:44 AM
dexonsmith committed rG0a2be46cfdb6: Modules: Invalidate out-of-date PCMs as they're discovered (authored by dexonsmith).
Modules: Invalidate out-of-date PCMs as they're discovered
Mar 9 2019, 9:44 AM
dexonsmith committed rL355778: Modules: Invalidate out-of-date PCMs as they're discovered.
Modules: Invalidate out-of-date PCMs as they're discovered
Mar 9 2019, 9:44 AM
dexonsmith committed rC355778: Modules: Invalidate out-of-date PCMs as they're discovered.
Modules: Invalidate out-of-date PCMs as they're discovered
Mar 9 2019, 9:44 AM
dexonsmith committed rC355777: Modules: Rename MemoryBufferCache to InMemoryModuleCache.
Modules: Rename MemoryBufferCache to InMemoryModuleCache
Mar 9 2019, 9:34 AM
dexonsmith committed rG8bef5cd49a8b: Modules: Rename MemoryBufferCache to InMemoryModuleCache (authored by dexonsmith).
Modules: Rename MemoryBufferCache to InMemoryModuleCache
Mar 9 2019, 9:34 AM
dexonsmith closed D58890: Modules: Rename MemoryBufferCache to InMemoryModuleCache.

Committed in r355777.

Mar 9 2019, 9:34 AM
dexonsmith committed rL355777: Modules: Rename MemoryBufferCache to InMemoryModuleCache.
Modules: Rename MemoryBufferCache to InMemoryModuleCache
Mar 9 2019, 9:33 AM
dexonsmith added a child revision for D58893: Modules: Invalidate out-of-date PCMs as they're discovered: D59176: Modules: Add LangOptions::CacheGeneratedPCH.
Mar 9 2019, 9:23 AM
dexonsmith added a parent revision for D59176: Modules: Add LangOptions::CacheGeneratedPCH: D58893: Modules: Invalidate out-of-date PCMs as they're discovered.
Mar 9 2019, 9:23 AM
dexonsmith created D59176: Modules: Add LangOptions::CacheGeneratedPCH.
Mar 9 2019, 9:23 AM

Mar 7 2019

dexonsmith added inline comments to D59112: [Bitcode] Fix bitcode compatibility issue with clang.arc.use intrinsic.
Mar 7 2019, 7:42 PM · Restricted Project
dexonsmith accepted D59112: [Bitcode] Fix bitcode compatibility issue with clang.arc.use intrinsic.

LGTM too with some nitpicks (forgot to hit "submit" earlier this afternoon).

Mar 7 2019, 7:29 PM · Restricted Project
dexonsmith resigned from D59080: Merge of global constants does not happen when constants have common linkage.

+ab, do you still know this code?

Mar 7 2019, 10:25 AM