Page MenuHomePhabricator

CodaFi (Robert Widmann)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 17 2017, 7:43 PM (91 w, 1 d)

Recent Activity

Mon, Jan 7

CodaFi committed rL350608: [LLVM-C] Allow For Creating a BasicBlock without a Parent Function.
[LLVM-C] Allow For Creating a BasicBlock without a Parent Function
Mon, Jan 7, 10:31 PM
CodaFi closed D56279: [LLVM-C] Allow For Creating a BasicBlock without a Parent Function.
Mon, Jan 7, 10:30 PM
CodaFi committed rL350607: [LLVM-C] Allow Specifying Signedness in Int Cast.
[LLVM-C] Allow Specifying Signedness in Int Cast
Mon, Jan 7, 10:30 PM
CodaFi closed D56280: [LLVM-C] Allow Specifying Signedness in Int Cast.
Mon, Jan 7, 10:30 PM

Thu, Jan 3

CodaFi created D56280: [LLVM-C] Allow Specifying Signedness in Int Cast.
Thu, Jan 3, 10:55 AM
CodaFi updated the diff for D56279: [LLVM-C] Allow For Creating a BasicBlock without a Parent Function.
Thu, Jan 3, 10:49 AM
CodaFi created D56279: [LLVM-C] Allow For Creating a BasicBlock without a Parent Function.
Thu, Jan 3, 10:27 AM

Wed, Jan 2

CodaFi committed rL350279: [LLVM-C] Expand LLVMRelocMode.
[LLVM-C] Expand LLVMRelocMode
Wed, Jan 2, 4:37 PM
CodaFi closed D56187: [LLVM-C] Expand LLVMRelocMode.
Wed, Jan 2, 4:37 PM

Tue, Jan 1

CodaFi created D56187: [LLVM-C] Expand LLVMRelocMode.
Tue, Jan 1, 11:12 AM
CodaFi committed rL350197: [LLVM-C] bool -> LLVMBool.
[LLVM-C] bool -> LLVMBool
Tue, Jan 1, 11:07 AM
CodaFi committed rL350196: [LLVM-C] Add Accessors for Discarding Value Names in the IR.
[LLVM-C] Add Accessors for Discarding Value Names in the IR
Tue, Jan 1, 11:01 AM
CodaFi closed D56179: [LLVM-C] Add Accessors for Discarding Value Names in the IR.
Tue, Jan 1, 11:01 AM

Mon, Dec 31

CodaFi created D56179: [LLVM-C] Add Accessors for Discarding Value Names in the IR.
Mon, Dec 31, 12:48 PM
CodaFi created D56177: [LLVM-C] Add Bindings to GlobalIFunc.
Mon, Dec 31, 12:03 PM

Nov 8 2018

CodaFi added a comment to D52239: [OCaml] Add OCaml APIs to access DebugLoc info.

@whitequark I meant to comment here, I'm not sure how I wound up on that other issue. Reposting for visibility:

Nov 8 2018, 1:32 PM

Nov 5 2018

CodaFi committed rL346197: [LLVM-C] Fix Windows Build of Core.
[LLVM-C] Fix Windows Build of Core
Nov 5 2018, 5:56 PM
CodaFi committed rL346195: [LLVM-C] Improve Intrinsics Bindings.
[LLVM-C] Improve Intrinsics Bindings
Nov 5 2018, 5:40 PM
CodaFi closed D53626: [LLVM-C] Improve Intrinsics Bindings.
Nov 5 2018, 5:40 PM
CodaFi added a comment to D53841: [OCaml] Fix incorrect use of CAMLlocal in nested blocks.

Apologies for not bringing this up in the earlier patch, but is there a reason this isn't using/building on the existing DI C bindings we have?

Nov 5 2018, 5:37 PM

Oct 29 2018

CodaFi updated the diff for D53626: [LLVM-C] Improve Intrinsics Bindings.
Oct 29 2018, 8:38 AM
CodaFi committed rL345508: [LLVM-C] Add Builder Bindings to Common Memory Intrinsics.
[LLVM-C] Add Builder Bindings to Common Memory Intrinsics
Oct 29 2018, 8:34 AM
CodaFi closed D53555: [LLVM-C] Add Builder Bindings to Common Memory Intrinsics.
Oct 29 2018, 8:34 AM

Oct 23 2018

CodaFi added a comment to D53555: [LLVM-C] Add Builder Bindings to Common Memory Intrinsics.

The infrastructure to test this isn't yet in place. I will merge accessors for MemSetBase and MemTransferBase in a later patch then special-case the echo test to call out to these build functions with those arguments. It's not a particularly attractive plan, but it should work fine.

Oct 23 2018, 5:49 PM
CodaFi created D53626: [LLVM-C] Improve Intrinsics Bindings.
Oct 23 2018, 5:36 PM

Oct 22 2018

CodaFi retitled D53555: [LLVM-C] Add Builder Bindings to Common Memory Intrinsics from [LLVM-C] Add Builder Bindings to Common Memory Intrinsic to [LLVM-C] Add Builder Bindings to Common Memory Intrinsics.
Oct 22 2018, 9:55 PM
CodaFi created D53555: [LLVM-C] Add Builder Bindings to Common Memory Intrinsics.
Oct 22 2018, 9:55 PM

Oct 1 2018

CodaFi committed rL343469: [LLVM-C] Add an accessor for the kind of a Metadata Node.
[LLVM-C] Add an accessor for the kind of a Metadata Node
Oct 1 2018, 6:17 AM
CodaFi closed D52693: [LLVM-C] Add an accessor for the kind of a Metadata Node.
Oct 1 2018, 6:17 AM

Sep 29 2018

CodaFi added a comment to D52693: [LLVM-C] Add an accessor for the kind of a Metadata Node.

@whitequark Done.

Sep 29 2018, 2:40 PM
CodaFi updated the diff for D52693: [LLVM-C] Add an accessor for the kind of a Metadata Node.
Sep 29 2018, 2:24 PM
CodaFi added inline comments to D52693: [LLVM-C] Add an accessor for the kind of a Metadata Node.
Sep 29 2018, 8:47 AM
CodaFi created D52693: [LLVM-C] Add an accessor for the kind of a Metadata Node.
Sep 29 2018, 8:38 AM

Sep 28 2018

CodaFi committed rL343363: [LLVM-C] Add an accessor for the "value type" of a global.
[LLVM-C] Add an accessor for the "value type" of a global
Sep 28 2018, 1:56 PM
CodaFi closed D52659: [LLVM-C] Add an accessor for the "value type" of a global.
Sep 28 2018, 1:56 PM
CodaFi updated the summary of D52659: [LLVM-C] Add an accessor for the "value type" of a global.
Sep 28 2018, 9:25 AM
CodaFi updated the summary of D52659: [LLVM-C] Add an accessor for the "value type" of a global.
Sep 28 2018, 9:25 AM
CodaFi created D52659: [LLVM-C] Add an accessor for the "value type" of a global.
Sep 28 2018, 9:24 AM
CodaFi committed rL343334: [LLVM-C] Fix broken build bots.
[LLVM-C] Fix broken build bots
Sep 28 2018, 9:05 AM
CodaFi closed D52657: [LLVM-C] Fix broken build bots.
Sep 28 2018, 9:05 AM
CodaFi created D52657: [LLVM-C] Fix broken build bots.
Sep 28 2018, 9:00 AM
CodaFi committed rL343330: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.
[LLVM-C] Add more debug information accessors to GlobalObject and Instruction
Sep 28 2018, 8:39 AM
CodaFi closed D51522: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.
Sep 28 2018, 8:39 AM
CodaFi abandoned D46166: [LLVM-C] Add accessor to retrieve DINode's tag.

Abandoning this revision to keep it out of the review queue.

Sep 28 2018, 8:35 AM
CodaFi added a comment to D51522: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.

@whitequark Ping

Sep 28 2018, 8:00 AM

Sep 10 2018

CodaFi added a comment to D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal..

The regression tests are only against the text of the driver's cc1 commands. It's not actually testing if we can compile.

Sep 10 2018, 5:36 PM
CodaFi added a comment to D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal..

@kristina They're having linker troubles because this driver is hardcoding an incorrect linker flag for libbuiltin on the arm-baremetal triple. D33259 introduced this behavior by smashing together the -lclang_rt.builtins-$ARCH flag with the proper resource-directory-relative absolute path <RESOURCE_DIR>/lib/libclang_rt.builtins-$ARCH.a which resulted in the incorrect flag -libclang_rt.builtins-$ARCH.a. With that path, the linker is going to try to look for "libclang_rt.builtins-$ARCH.a.a" in the link libraries search paths. The driver should be forming an exact path to this library relative to the resource directory. If users wish to override this behavior, they need to pass -nostdlib or -nodefaultlibs

Sep 10 2018, 5:32 PM
CodaFi added a comment to D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal..

Two things:

Sep 10 2018, 5:11 PM

Aug 31 2018

CodaFi updated the diff for D51522: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.
Aug 31 2018, 10:22 AM

Aug 30 2018

CodaFi added a comment to D51522: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.

How is a user of this API supposed to map the unsigned Kind values to something meaningful? We don't have an enum LLVMMetadataKind, do we?

Aug 30 2018, 8:40 PM
CodaFi created D51522: [LLVM-C] Add more debug information accessors to GlobalObject and Instruction.
Aug 30 2018, 4:03 PM
CodaFi committed rL341085: [LLVM-C] Add Bindings For Named Metadata.
[LLVM-C] Add Bindings For Named Metadata
Aug 30 2018, 10:13 AM
CodaFi closed D47179: [LLVM-C] Add Bindings For Named Metadata.
Aug 30 2018, 10:12 AM
CodaFi added a comment to D47179: [LLVM-C] Add Bindings For Named Metadata.

@whitequark Could I get another once-over on the C API here?

Aug 30 2018, 9:46 AM

Aug 27 2018

CodaFi updated the diff for D47179: [LLVM-C] Add Bindings For Named Metadata.
Aug 27 2018, 11:44 AM
CodaFi added inline comments to D47179: [LLVM-C] Add Bindings For Named Metadata.
Aug 27 2018, 11:36 AM
CodaFi added inline comments to D47179: [LLVM-C] Add Bindings For Named Metadata.
Aug 27 2018, 10:21 AM

Aug 25 2018

CodaFi updated the diff for D47179: [LLVM-C] Add Bindings For Named Metadata.

Spoke with @aprantl in person about this, and we came to the conclusion that named metadata nodes are different enough that the original TODO doesn't make any sense. For one, it would mean that you could use named metadata nodes to create more kinds of cycles. It would also mean the verifier would have to be extended to reject any usage of named metadata nodes as operand metadata.

Aug 25 2018, 4:33 PM
CodaFi updated the summary of D47179: [LLVM-C] Add Bindings For Named Metadata.
Aug 25 2018, 4:25 PM
CodaFi committed rL340691: [C-API][DIBuilder] Use NameLen in LLVMDIBuilderCreateParameterVariable.
[C-API][DIBuilder] Use NameLen in LLVMDIBuilderCreateParameterVariable
Aug 25 2018, 12:55 PM
CodaFi closed D51141: [C-API][DIBuilder] Use NameLen in LLVMDIBuilderCreateParameterVariable.
Aug 25 2018, 12:55 PM

Aug 23 2018

CodaFi accepted D51141: [C-API][DIBuilder] Use NameLen in LLVMDIBuilderCreateParameterVariable.

Thanks for catching this! On top of respecting our API contract, it's good to be safe around C strings.

Aug 23 2018, 2:10 PM

May 21 2018

CodaFi added a comment to D47179: [LLVM-C] Add Bindings For Named Metadata.

@aprantl Let me know if this is the right way to address that FIXME. There didn't seem to be any consequences of the small change I've got here which could be very good or very bad from my naive perspective.

May 21 2018, 10:37 PM
CodaFi created D47179: [LLVM-C] Add Bindings For Named Metadata.
May 21 2018, 10:36 PM
CodaFi committed rL332850: [LLVM-C] Add DIBuilder Bindings For ObjC Classes.
[LLVM-C] Add DIBuilder Bindings For ObjC Classes
May 21 2018, 9:31 AM
CodaFi closed D47123: [LLVM-C] Add DIBuilder Bindings For ObjC Classes.
May 21 2018, 9:31 AM

May 20 2018

CodaFi committed rL332826: [LLVM-C] Improve Bindings For Aliases.
[LLVM-C] Improve Bindings For Aliases
May 20 2018, 4:53 PM
CodaFi closed D46808: [LLVM-C] Improve Bindings For Aliases.
May 20 2018, 4:53 PM
CodaFi created D47123: [LLVM-C] Add DIBuilder Bindings For ObjC Classes.
May 20 2018, 4:51 PM
CodaFi updated the diff for D46808: [LLVM-C] Improve Bindings For Aliases.
May 20 2018, 10:31 AM

May 19 2018

CodaFi committed rL332810: [LLVM-C] Use Length-Providing Value Name Getters and Setters.
[LLVM-C] Use Length-Providing Value Name Getters and Setters
May 19 2018, 8:13 AM
CodaFi closed D46890: [LLVM-C] Use Length-Providing Value Name Getters and Setters.
May 19 2018, 8:13 AM

May 18 2018

CodaFi updated the summary of D46890: [LLVM-C] Use Length-Providing Value Name Getters and Setters.
May 18 2018, 7:25 PM
CodaFi updated the diff for D46890: [LLVM-C] Use Length-Providing Value Name Getters and Setters.
May 18 2018, 6:56 PM
CodaFi added a comment to D46890: [LLVM-C] Use Length-Providing Value Name Getters and Setters.

First, the naming here is absurdly confusing. Second, it's also inconsistent with all other functions, e.g. LLVMGetSourceFileName/LLVMSetSourceFileName.

Use LLVMGetValueName2 and LLVMSetValueName2, since this is what we do when updating function signatures in the C API.

May 18 2018, 6:45 PM

May 15 2018

CodaFi created D46890: [LLVM-C] Use Length-Providing Value Name Getters and Setters.
May 15 2018, 9:32 AM

May 14 2018

CodaFi committed rL332219: [LLVM-C] Add Bindings For Module Flags.
[LLVM-C] Add Bindings For Module Flags
May 14 2018, 1:12 AM
CodaFi closed D46792: [LLVM-C] Add Bindings For Module Flags.
May 14 2018, 1:12 AM

May 13 2018

CodaFi updated the diff for D46792: [LLVM-C] Add Bindings For Module Flags.
May 13 2018, 10:56 PM
CodaFi added inline comments to D46808: [LLVM-C] Improve Bindings For Aliases.
May 13 2018, 10:51 PM
CodaFi added inline comments to D46792: [LLVM-C] Add Bindings For Module Flags.
May 13 2018, 10:49 PM
CodaFi created D46808: [LLVM-C] Improve Bindings For Aliases.
May 13 2018, 12:08 PM

May 12 2018

CodaFi updated the diff for D46792: [LLVM-C] Add Bindings For Module Flags.
May 12 2018, 11:39 AM

May 11 2018

CodaFi updated the diff for D46792: [LLVM-C] Add Bindings For Module Flags.
May 11 2018, 9:21 PM
CodaFi created D46792: [LLVM-C] Add Bindings For Module Flags.
May 11 2018, 9:17 PM

May 10 2018

CodaFi committed rL332041: [LLVM-C] Consolidate llgo's DIBuilder Bindings.
[LLVM-C] Consolidate llgo's DIBuilder Bindings
May 10 2018, 2:14 PM
CodaFi closed D46725: [LLVM-C] Consolidate llgo's DIBuilder Bindings.
May 10 2018, 2:13 PM
CodaFi created D46725: [LLVM-C] Consolidate llgo's DIBuilder Bindings.
May 10 2018, 2:02 PM
CodaFi committed rL332014: [LLVM-C] Add Accessors for Common DIType and DILocation Properties.
[LLVM-C] Add Accessors for Common DIType and DILocation Properties
May 10 2018, 11:27 AM
CodaFi closed D46627: [LLVM-C] Add Accessors for Common DIType and DILocation Properties.
May 10 2018, 11:27 AM
CodaFi added a comment to D46627: [LLVM-C] Add Accessors for Common DIType and DILocation Properties.

You could write the same tests as in llvm-c-test/echo.cpp. But I would not block merging on this.

May 10 2018, 11:22 AM
CodaFi committed rL332010: [LLVM-C] Move DIBuilder Bindings For Temporary MDNodes.
[LLVM-C] Move DIBuilder Bindings For Temporary MDNodes
May 10 2018, 11:13 AM
CodaFi closed D46632: [LLVM-C] Move DIBuilder Bindings For Temporary MDNodes.
May 10 2018, 11:13 AM

May 9 2018

CodaFi added a comment to D46627: [LLVM-C] Add Accessors for Common DIType and DILocation Properties.

Any tests?

May 9 2018, 8:22 PM
CodaFi added a comment to D46632: [LLVM-C] Move DIBuilder Bindings For Temporary MDNodes.

llvm::Value::replaceAllUsesWith and llvm::MDNode::replaceAllUsesWith are distinct operations. The former assumes the source and replacement values are the same type, the latter assumes that you're trying to build up a graph of dummy metadata for an incomplete or recursive definition that you will eventually finalize by replacing it with real nodes.

May 9 2018, 8:19 PM
CodaFi updated the diff for D46632: [LLVM-C] Move DIBuilder Bindings For Temporary MDNodes.
May 9 2018, 8:18 PM
CodaFi created D46632: [LLVM-C] Move DIBuilder Bindings For Temporary MDNodes.
May 9 2018, 12:12 AM

May 8 2018

CodaFi committed rL331850: [LLVM-C] Correct types in Go bindings.
[LLVM-C] Correct types in Go bindings
May 8 2018, 11:49 PM
CodaFi closed D46631: [LLVM-C] Correct types in Go bindings.
May 8 2018, 11:49 PM
CodaFi created D46631: [LLVM-C] Correct types in Go bindings.
May 8 2018, 11:47 PM