fjricci (Francis Ricci)
User

Projects

User does not belong to any projects.

User Details

User Since
Dec 4 2015, 11:34 AM (102 w, 2 d)

Recent Activity

Thu, Nov 16

fjricci accepted D40143: [sanitizer] Refactor how assembly files are handled.

Lgtm

Thu, Nov 16, 11:41 AM · Restricted Project

Wed, Nov 15

fjricci committed rL318336: Fix some typos in a comment..
Fix some typos in a comment.
Wed, Nov 15, 1:19 PM
fjricci accepted D40052: [asan] Allow getpwnam(NULL) for binary compatibily.
Wed, Nov 15, 1:14 PM · Restricted Project
fjricci accepted D40053: [asan] Don't crash on fclose(NULL).
Wed, Nov 15, 1:14 PM · Restricted Project

Tue, Nov 14

fjricci accepted D39987: [compiler-rt] Switch from deprecated TARGET_IPHONE_SIMULATOR to TARGET_OS_SIMULATOR.
Tue, Nov 14, 7:57 AM · Restricted Project

Fri, Nov 10

fjricci added a comment to D39888: [lsan] Only use VM_MEMORY_OS_ALLOC_ONCE on Darwin versions that support it.

An alternative could be to just disable the build of LSan on OS X < 10.9 (or even 10.11)

Fri, Nov 10, 7:40 AM
fjricci requested changes to D39888: [lsan] Only use VM_MEMORY_OS_ALLOC_ONCE on Darwin versions that support it.

Because macOS changes quite a bit from version to version, we (or at least I) don't have strong plans to support LSan outside of 10.11+. Not sure what darwin used to store singleton kernel allocations before this page was introduced, but I assume that just not handling those allocations will lead to a ton of false positives. I'm inclined to let this fail on old versions unless there's someone willing to do the work to make sure they're properly supported. cc @kubamracek

Fri, Nov 10, 7:38 AM

Thu, Nov 9

fjricci accepted D39861: [compiler-rt] Fix const and volatile qualifier warnings.

lgtm

Thu, Nov 9, 4:03 PM · Restricted Project

Mon, Oct 23

fjricci added a comment to D27607: [ubsan] Treat ObjC's BOOL as if its range is always {0, 1}.

Awesome, good to know. Thanks!

Mon, Oct 23, 11:52 AM
fjricci added inline comments to D39151: [tsan] Fix warnings in tsan_interceptors.cc from expansion of variadic macros.
Mon, Oct 23, 10:12 AM
fjricci added inline comments to D39151: [tsan] Fix warnings in tsan_interceptors.cc from expansion of variadic macros.
Mon, Oct 23, 8:16 AM

Sat, Oct 21

fjricci added reviewers for D39151: [tsan] Fix warnings in tsan_interceptors.cc from expansion of variadic macros: kubamracek, alekseyshl.
Sat, Oct 21, 8:16 AM

Oct 19 2017

fjricci added a comment to D27607: [ubsan] Treat ObjC's BOOL as if its range is always {0, 1}.

On platforms where BOOL == signed char, is it actually undefined behavior (or is it just bad programming practice) to store a value other than 0 or 1 in your BOOL? I can't find any language specs suggesting that it is, and given that it's just a typedef for a signed char, I don't see why it would be.

Oct 19 2017, 5:42 PM

Oct 16 2017

fjricci added a comment to D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.

(for posterity, the build is not failing anymore with this patch committed)

Oct 16 2017, 6:55 PM
fjricci added a comment to D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.

Failing build: http://green.lab.llvm.org/green/job/clang-stage2-cmake-RgSan/5341/

Oct 16 2017, 6:54 PM

Oct 10 2017

fjricci committed rL315338: [lit] Add host_cxx to lit config.
[lit] Add host_cxx to lit config
Oct 10 2017, 10:54 AM
fjricci closed D38741: [lit] Add host_cxx to lit config by committing rL315338: [lit] Add host_cxx to lit config.
Oct 10 2017, 10:54 AM
fjricci committed rL315337: [llvm-objdump] Disable leak checking on an llvm-objdump test.
[llvm-objdump] Disable leak checking on an llvm-objdump test
Oct 10 2017, 10:51 AM
fjricci closed D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test by committing rL315337: [llvm-objdump] Disable leak checking on an llvm-objdump test.
Oct 10 2017, 10:51 AM
fjricci added a comment to D38703: [lit] Only enable LSan on darwin when clang supports it.

@qcolombet - Can you keep an eye on the internal apple builders and let me know if this doesn't fix them? I'll be out of the town the rest of the week though, so I probably won't be able to do much until Monday.

Oct 10 2017, 10:30 AM
fjricci committed rL315333: [lit] Only enable LSan on darwin when clang supports it.
[lit] Only enable LSan on darwin when clang supports it
Oct 10 2017, 10:21 AM
fjricci closed D38703: [lit] Only enable LSan on darwin when clang supports it by committing rL315333: [lit] Only enable LSan on darwin when clang supports it.
Oct 10 2017, 10:21 AM
fjricci updated the diff for D38703: [lit] Only enable LSan on darwin when clang supports it.

Improve warning messages

Oct 10 2017, 10:10 AM
fjricci added a comment to D38741: [lit] Add host_cxx to lit config.

Used here: https://reviews.llvm.org/D38703

Oct 10 2017, 9:58 AM
fjricci added a reviewer for D38741: [lit] Add host_cxx to lit config: ddunbar.
Oct 10 2017, 9:20 AM
fjricci added a comment to D38703: [lit] Only enable LSan on darwin when clang supports it.

Clang change here: reviews.llvm.org/D38741

Oct 10 2017, 9:19 AM
fjricci created D38741: [lit] Add host_cxx to lit config.
Oct 10 2017, 9:19 AM
fjricci updated the diff for D38703: [lit] Only enable LSan on darwin when clang supports it.

Add warnings

Oct 10 2017, 9:16 AM
fjricci added a comment to D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.

I'm going to be on vacation starting tomorrow (oct 11), and this seems like a pretty unoffensive change to me, so if I don't hear anything positive or negative by the end of the day, I'll commit this to fix the green bootstrapped builders.

Oct 10 2017, 9:01 AM
fjricci added a comment to D38703: [lit] Only enable LSan on darwin when clang supports it.

I'll upload the clang host_cxx patch shortly, but this shouldn't be blocked on that (since it handles the undefined case).

Oct 10 2017, 8:58 AM
fjricci updated the diff for D38703: [lit] Only enable LSan on darwin when clang supports it.

Address comments

Oct 10 2017, 8:57 AM
fjricci added a reviewer for D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test: enderby.
Oct 10 2017, 7:21 AM
fjricci added a comment to D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.

Fairly certain that if this is 10.11-specific, it's either a leak in libxar, or a bug in LeakSanitizer. It doesn't appear to be a leak in objdump itself (particularly when looking at the stack, which has a dispatch_once call in it)

Oct 10 2017, 7:20 AM

Oct 9 2017

fjricci added inline comments to D38703: [lit] Only enable LSan on darwin when clang supports it.
Oct 9 2017, 3:40 PM
fjricci added inline comments to D38703: [lit] Only enable LSan on darwin when clang supports it.
Oct 9 2017, 3:38 PM
fjricci added inline comments to D38703: [lit] Only enable LSan on darwin when clang supports it.
Oct 9 2017, 2:59 PM
fjricci added inline comments to D38703: [lit] Only enable LSan on darwin when clang supports it.
Oct 9 2017, 2:52 PM
fjricci added inline comments to D38598: [llvm-objdump] Add RAII for xar apis.
Oct 9 2017, 1:29 PM
fjricci committed rL315243: [llvm-objdump] Use initializer list for scoped xar api constructors.
[llvm-objdump] Use initializer list for scoped xar api constructors
Oct 9 2017, 1:29 PM
fjricci updated the diff for D38703: [lit] Only enable LSan on darwin when clang supports it.

Fix a couple python issues

Oct 9 2017, 1:20 PM
fjricci planned changes to D38703: [lit] Only enable LSan on darwin when clang supports it.

Fixing a couple issues.

Oct 9 2017, 1:16 PM
fjricci created D38703: [lit] Only enable LSan on darwin when clang supports it.
Oct 9 2017, 1:07 PM
fjricci added a comment to D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.

I got this failure to reproduce locally on my macOS 10.11 VM, but it's probably still worth disabling the leak checking on this test to fix the buildbots until I can diagnose the issue.

Oct 9 2017, 12:32 PM
fjricci created D38699: [llvm-objdump] Disable leak checking on an llvm-objdump test.
Oct 9 2017, 11:34 AM
fjricci committed rL315218: [dsymutil] Emit valid debug locations when no symbol flags are set.
[dsymutil] Emit valid debug locations when no symbol flags are set
Oct 9 2017, 10:29 AM
fjricci closed D38587: [dsymutil] Emit valid debug locations when no symbol flags are set by committing rL315218: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 9 2017, 10:29 AM
fjricci added a comment to D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

Oh interesting. I tried using PWD=/, but I didn't try actually cd-ing into that directory. I'll try that - thanks!

Oct 9 2017, 9:29 AM
fjricci accepted D38686: [dsymutil] Timestmap verification for __swift_ast.

Looks good to me

Oct 9 2017, 7:41 AM
fjricci added a comment to D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

I'm running into some issues with using a precompiled binary + object file here. For some reason, the debug info contains an absolute path to the object file instead of a relative path, so dsymutil will fail because it's looking for a path that doesn't exist (ie a path on my local machine). This is true even if I use -fdebug-compilation-dir (it updates the debug info in the IR, but doesn't change where dsymutil looks for the object file). I also tried -add_ast_path, which works as a hacky workaround because it will search the relative path in addition to the absolute path, but that seems like a bad solution.

Oct 9 2017, 7:29 AM

Oct 6 2017

fjricci updated the diff for D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

Use precompiled binary with IR as a comment to allow testing on linux

Oct 6 2017, 11:31 AM
fjricci reopened D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 6 2017, 11:30 AM
fjricci committed rL315088: Revert "[dsymutil] Emit valid debug locations when no symbol flags are set".
Revert "[dsymutil] Emit valid debug locations when no symbol flags are set"
Oct 6 2017, 11:21 AM
fjricci committed rL315082: [dsymutil] Emit valid debug locations when no symbol flags are set.
[dsymutil] Emit valid debug locations when no symbol flags are set
Oct 6 2017, 10:45 AM
fjricci closed D38587: [dsymutil] Emit valid debug locations when no symbol flags are set by committing rL315082: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 6 2017, 10:45 AM
fjricci committed rL315072: Guard xar RAII behind HAVE_LIBXAR.
Guard xar RAII behind HAVE_LIBXAR
Oct 6 2017, 8:56 AM
fjricci committed rL315069: [llvm-objdump] Add RAII for xar apis.
[llvm-objdump] Add RAII for xar apis
Oct 6 2017, 8:35 AM
fjricci closed D38598: [llvm-objdump] Add RAII for xar apis by committing rL315069: [llvm-objdump] Add RAII for xar apis.
Oct 6 2017, 8:35 AM
fjricci updated the diff for D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

Move to x86 test dir

Oct 6 2017, 8:05 AM
fjricci committed rL315066: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
[llvm-dsymutil] Add support for __swift_ast MachO DWARF section
Oct 6 2017, 7:51 AM
fjricci closed D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section by committing rL315066: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
Oct 6 2017, 7:51 AM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

clang-format

Oct 6 2017, 7:41 AM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

Delete copy constructor (which implicitly deletes move constructor), and
assignment operator

Oct 6 2017, 7:38 AM

Oct 5 2017

fjricci added a comment to D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

They fail with:
while processing dwarf streamer init
error: : error: unable to get target for 'x86_64-apple-darwin', see --version and --triple.

Oct 5 2017, 4:17 PM
fjricci reopened D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Weirdly, appending .macho.x86_64 fixed the armv7/thumbv7 buildbots, but not aarch64. Will try to diagnose.

Oct 5 2017, 4:16 PM
fjricci committed rL315034: Revert "[llvm-dsymutil] Add support for __swift_ast MachO DWARF section".
Revert "[llvm-dsymutil] Add support for __swift_ast MachO DWARF section"
Oct 5 2017, 4:11 PM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

Don't call deallocation routines on null xar objects

Oct 5 2017, 4:09 PM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

anonymous namespace

Oct 5 2017, 2:21 PM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

Xar -> ScopedXar

Oct 5 2017, 2:06 PM
fjricci updated the diff for D38598: [llvm-objdump] Add RAII for xar apis.

clang-format

Oct 5 2017, 1:58 PM
fjricci updated subscribers of D38598: [llvm-objdump] Add RAII for xar apis.
Oct 5 2017, 1:57 PM
fjricci created D38598: [llvm-objdump] Add RAII for xar apis.
Oct 5 2017, 1:55 PM
fjricci committed rL315017: [dsymutil] Fix typo in swift-ast.test.
[dsymutil] Fix typo in swift-ast.test
Oct 5 2017, 1:18 PM
fjricci committed rL315014: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
[llvm-dsymutil] Add support for __swift_ast MachO DWARF section
Oct 5 2017, 1:05 PM
fjricci closed D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section by committing rL315014: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
Oct 5 2017, 1:04 PM
fjricci added a comment to D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

I think that postfixing the binary name with ".macho.x86_64" will work, that seems to be what the other tests do, although I can't seem to find a lit config relating to that.

Oct 5 2017, 12:58 PM
fjricci reopened D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

The test fails on non apple-x86_64 builders, reverted the patch for now

Oct 5 2017, 12:52 PM
fjricci committed rL315009: Revert "[llvm-dsymutil] Add support for __swift_ast MachO DWARF section".
Revert "[llvm-dsymutil] Add support for __swift_ast MachO DWARF section"
Oct 5 2017, 12:49 PM
fjricci committed rL315006: [dsymutil] Fix unused variable warning.
[dsymutil] Fix unused variable warning
Oct 5 2017, 12:37 PM
fjricci committed rL315004: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
[llvm-dsymutil] Add support for __swift_ast MachO DWARF section
Oct 5 2017, 12:19 PM
fjricci closed D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section by committing rL315004: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.
Oct 5 2017, 12:19 PM
fjricci updated the diff for D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

Maintain same behavior as apple's dsymutil

Oct 5 2017, 12:05 PM
fjricci added inline comments to D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 5 2017, 9:33 AM
fjricci added inline comments to D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 5 2017, 9:31 AM
fjricci updated the diff for D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

Prune some unneeded IR and debug info

Oct 5 2017, 8:48 AM
fjricci planned changes to D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.

I'm going to try to clean up the IR a bit

Oct 5 2017, 8:34 AM
fjricci created D38587: [dsymutil] Emit valid debug locations when no symbol flags are set.
Oct 5 2017, 7:54 AM
fjricci updated the diff for D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Address comments

Oct 5 2017, 7:35 AM

Oct 4 2017

fjricci updated the diff for D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Fix -no-output mode

Oct 4 2017, 1:44 PM
fjricci updated the diff for D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Clean up comments

Oct 4 2017, 1:32 PM
fjricci updated the diff for D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Asm->OutStreamer -> MS

Oct 4 2017, 1:20 PM
fjricci committed rL314915: [test] Fix append_path in the empty case.
[test] Fix append_path in the empty case
Oct 4 2017, 10:32 AM
fjricci closed D38542: [test] Fix append_path in the empty case by committing rL314915: [test] Fix append_path in the empty case.
Oct 4 2017, 10:32 AM
fjricci updated the diff for D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Add test

Oct 4 2017, 10:31 AM
fjricci added a comment to D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

Looks like they check in binaries as well. I'll upload a version with a binary for now, and I can swap it out if we decide we want to.

Oct 4 2017, 10:29 AM
fjricci added a comment to D38504: [llvm-dsymutil] Add support for __swift_ast MachO DWARF section.

I have a test case working with an IR file, but it looks like most of the existing dsymutil tests just use an object file - should I do that here as well? (I don't think we get much value from using an IR file over an object file)

Oct 4 2017, 10:05 AM
fjricci updated the diff for D38542: [test] Fix append_path in the empty case.

fix append_path

Oct 4 2017, 9:39 AM
fjricci planned changes to D38542: [test] Fix append_path in the empty case.
Oct 4 2017, 9:32 AM
fjricci retitled D38542: [test] Fix append_path in the empty case from [test] Fix ASAN_OPTIONS to prevent adding trailing semicolon to [test] Fix ASAN_OPTIONS to prevent adding trailing colon.
Oct 4 2017, 8:15 AM
fjricci updated the summary of D38542: [test] Fix append_path in the empty case.
Oct 4 2017, 8:15 AM
fjricci created D38542: [test] Fix append_path in the empty case.
Oct 4 2017, 7:54 AM