Page MenuHomePhabricator

wjakob (wenzel.jakob@epfl.ch)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 10 2017, 8:52 AM (304 w, 1 d)

Recent Activity

Dec 7 2022

wjakob added a comment to D139381: Added a C-API binding to query the LLVM version.

Thank you. Please use

Dec 7 2022, 2:04 AM · Restricted Project, Restricted Project
wjakob updated the diff for D139381: Added a C-API binding to query the LLVM version.

Changed variable case

Dec 7 2022, 1:04 AM · Restricted Project, Restricted Project
wjakob updated the diff for D139381: Added a C-API binding to query the LLVM version.

Removed superfluous newline.

Dec 7 2022, 1:02 AM · Restricted Project, Restricted Project

Dec 6 2022

wjakob updated the diff for D139381: Added a C-API binding to query the LLVM version.

Thanks, that sounds more reasonable. I closed the other differential and updated this one with a squashed commit.

Dec 6 2022, 11:54 PM · Restricted Project, Restricted Project
wjakob abandoned D139460: added a test case.
Dec 6 2022, 11:52 PM · Restricted Project, Restricted Project
wjakob added a comment to D139381: Added a C-API binding to query the LLVM version.

I added a testcase and ran arcanist again, which created a child differential D139460. I hope that's right.

Dec 6 2022, 2:19 PM · Restricted Project, Restricted Project
wjakob requested review of D139460: added a test case.
Dec 6 2022, 2:15 PM · Restricted Project, Restricted Project

Dec 5 2022

wjakob added a comment to D139381: Added a C-API binding to query the LLVM version.

Dear LLVM developers -- this is my first contribution to the LLVM project, I hope I got it right.

Dec 5 2022, 5:17 PM · Restricted Project, Restricted Project
wjakob requested review of D139381: Added a C-API binding to query the LLVM version.
Dec 5 2022, 5:12 PM · Restricted Project, Restricted Project

Apr 10 2017

wjakob added a comment to D31885: Remove TBAA information from LValues representing union members.

Disclaimer: I am not an LLVM developer. Just looking at the patch, I wonder if it could be much less intrusive if LValue CodeGenFunction::EmitLValue(const Expr *E) is split into two methods -- a private one, and a public-facing one that applies your ClearTBAA method.

Apr 10 2017, 10:49 AM
wjakob added a comment to D31885: Remove TBAA information from LValues representing union members.

FYI: I gave it a try, and it looks like the union aliasing-related issue mentioned in Bug 32056 is not resolved by this patch.

Apr 10 2017, 8:55 AM