Page MenuHomePhabricator

erichkeane (Erich Keane)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 28 2016, 8:37 AM (137 w, 6 d)

Recent Activity

Fri, Feb 8

erichkeane accepted D57961: [X86] Add explicit alignment to __m128/__m128i/__m128d/etc. to allow matching of MSVC behavior with #pragma pack..

The justification is correct based on my research into this problem, and the code changes themselves look correct. I cannot think of any reason why re-stating the alignment will matter in GCC mode, so I think this is OK.

Fri, Feb 8, 10:09 AM · Restricted Project

Thu, Feb 7

erichkeane committed rG892e63319497: Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS. (authored by erichkeane).
Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS.
Thu, Feb 7, 7:14 AM
erichkeane committed rL353411: Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS..
Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS.
Thu, Feb 7, 7:14 AM
erichkeane committed rC353411: Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS..
Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS.
Thu, Feb 7, 7:14 AM

Fri, Feb 1

erichkeane added a comment to D57581: Explicitly add language standard option to test cases that rely on the C++14 default.

This seems innocuous to me. I don't see any problem with it, but please don't commit until the others have had time to look.

Fri, Feb 1, 5:56 AM · Restricted Project, Restricted Project

Tue, Jan 29

erichkeane accepted D57264: Bump minimum toolchain version.
Tue, Jan 29, 9:37 AM · Restricted Project

Fri, Jan 25

erichkeane accepted D57249: [clang-tidy] fix unit tests for dropped _Float16 support in X86.

LGTM, thanks for doing this!

Fri, Jan 25, 10:54 AM
erichkeane committed rL352229: Remove F16 literal support based on Float16 support..
Remove F16 literal support based on Float16 support.
Fri, Jan 25, 10:36 AM
erichkeane committed rC352229: Remove F16 literal support based on Float16 support..
Remove F16 literal support based on Float16 support.
Fri, Jan 25, 10:36 AM
erichkeane committed rC352222: Fix incorrect indent from r352221.
Fix incorrect indent from r352221
Fri, Jan 25, 9:40 AM
erichkeane committed rL352222: Fix incorrect indent from r352221.
Fix incorrect indent from r352221
Fri, Jan 25, 9:40 AM
erichkeane committed rC352221: Disable _Float16 for non ARM/SPIR Targets.
Disable _Float16 for non ARM/SPIR Targets
Fri, Jan 25, 9:30 AM
erichkeane committed rL352221: Disable _Float16 for non ARM/SPIR Targets.
Disable _Float16 for non ARM/SPIR Targets
Fri, Jan 25, 9:30 AM
erichkeane closed D57188: Disable _Float16 for non ARM/SPIR Targets.
Fri, Jan 25, 9:30 AM
erichkeane committed rL352219: Allow 'static' storage specifier on an out-of-line member function template.
Allow 'static' storage specifier on an out-of-line member function template
Fri, Jan 25, 9:02 AM
erichkeane committed rC352219: Allow 'static' storage specifier on an out-of-line member function template.
Allow 'static' storage specifier on an out-of-line member function template
Fri, Jan 25, 9:02 AM
erichkeane closed D56473: Allow 'static' storage specifier on an out-of-line member function template declaration in MSVCCompat mode.
Fri, Jan 25, 9:02 AM
erichkeane updated the diff for D57188: Disable _Float16 for non ARM/SPIR Targets.
Fri, Jan 25, 7:02 AM
erichkeane added inline comments to D57188: Disable _Float16 for non ARM/SPIR Targets.
Fri, Jan 25, 7:02 AM

Thu, Jan 24

erichkeane created D57188: Disable _Float16 for non ARM/SPIR Targets.
Thu, Jan 24, 1:45 PM
erichkeane committed rC352054: [CPU-Dispatch] Make pentium_iii_no_xmm_regs and pentium_iii alias..
[CPU-Dispatch] Make pentium_iii_no_xmm_regs and pentium_iii alias.
Thu, Jan 24, 7:29 AM
erichkeane committed rL352054: [CPU-Dispatch] Make pentium_iii_no_xmm_regs and pentium_iii alias..
[CPU-Dispatch] Make pentium_iii_no_xmm_regs and pentium_iii alias.
Thu, Jan 24, 7:29 AM
erichkeane added a comment to D54653: [IRBuilder] Remove positivity check from CreateAlignmentAssumption().

@erichkeane are you still going to land this?
If you don't want to bother with updating my tests, want me to take over so this doesn't get abandoned?

Thu, Jan 24, 6:04 AM

Jan 18 2019

erichkeane committed rC351576: [NFC] Fix wParentheses warning in ASTContext getFixedPointSemantics.
[NFC] Fix wParentheses warning in ASTContext getFixedPointSemantics
Jan 18 2019, 11:35 AM
erichkeane committed rL351576: [NFC] Fix wParentheses warning in ASTContext getFixedPointSemantics.
[NFC] Fix wParentheses warning in ASTContext getFixedPointSemantics
Jan 18 2019, 11:35 AM
erichkeane added inline comments to D56819: Document toolchain update policy.
Jan 18 2019, 10:00 AM
erichkeane added a comment to D45978: dllexport const variables must have external linkage..

@rnk and @aaron.ballman, does this look appropriate here?

Jan 18 2019, 9:26 AM
erichkeane added a reviewer for D45978: dllexport const variables must have external linkage.: aaron.ballman.
Jan 18 2019, 9:25 AM
erichkeane added a comment to rL351495: Make integral-o-pointer conversions in SFINAE illegal..

Taking a look now, thank you for letting me know!

Jan 18 2019, 6:02 AM
erichkeane committed rL351550: Fix test failure from r351495.
Fix test failure from r351495
Jan 18 2019, 6:02 AM
erichkeane committed rC351550: Fix test failure from r351495.
Fix test failure from r351495
Jan 18 2019, 6:01 AM
erichkeane added a comment to rL351495: Make integral-o-pointer conversions in SFINAE illegal..

Taking a look now, thank you for letting me know!

Jan 18 2019, 5:57 AM

Jan 17 2019

erichkeane committed rC351495: Make integral-o-pointer conversions in SFINAE illegal..
Make integral-o-pointer conversions in SFINAE illegal.
Jan 17 2019, 3:15 PM
erichkeane committed rL351495: Make integral-o-pointer conversions in SFINAE illegal..
Make integral-o-pointer conversions in SFINAE illegal.
Jan 17 2019, 3:15 PM
erichkeane accepted D56819: Document toolchain update policy.
Jan 17 2019, 10:28 AM

Jan 16 2019

erichkeane added a comment to D56799: [NFC] Factor out + document build requirements.

I think this is a good start. That said, this is also a bug-fix (not exactly NFC), since it adds the Clang check to AppleClang.

Jan 16 2019, 1:35 PM
erichkeane abandoned D47073: Document and Enforce new Host Compiler Policy.

This patch has run its course.

Jan 16 2019, 9:17 AM
erichkeane committed rL351357: Remove misleading line about git's lack of revision numbers..
Remove misleading line about git's lack of revision numbers.
Jan 16 2019, 9:05 AM

Jan 15 2019

erichkeane added a comment to D47073: Document and Enforce new Host Compiler Policy.

For what it's worth, the stricter version of this patch received quite overwhelming support in May. We delayed it due to Google's limitations and yet we still don't seem to have made progress. This current patch is a result of a continued set of compromises to try to satisfy a few in the vocal minority. I'd suggest that we all agree to have an ACTUAL decision made on llvm-dev, rather than this constant state of consensus derailed by vocal minority.

This *patch* does not matter to Google in any real sense that I'm aware of.... *All* of what I wrote in my latest comment is about this patch, and has nothing to do with the overall strategy of the LLVM project. And that feedback was given as a very active individual contributor to the project.

Does Google care a great deal about the overall state of host compiler support and C++ version support of LLVM? Yes, it does. We expressed that, and so did others. That was done on the -dev list as well as in the BoF. I'm not really interested in re-arguing it here.

But please try to take my feedback on this *patch* as just that. It is code review and feedback on the specific changes you are making. I'm genuinely trying to help get the CMake and documentation implementation *right* and useful for the project and the community, not for Google. I'm sorry that you're frustrated, I really am. But I am actually trying to help, and comments like this, IMO, are not constructive.

Jan 15 2019, 9:45 PM
erichkeane added a comment to D47073: Document and Enforce new Host Compiler Policy.

For what it's worth, the stricter version of this patch received quite overwhelming support in May. We delayed it due to Google's limitations and yet we still don't seem to have made progress. This current patch is a result of a continued set of compromises to try to satisfy a few in the vocal minority. I'd suggest that we all agree to have an ACTUAL decision made on llvm-dev, rather than this constant state of consensus derailed by vocal minority.

Jan 15 2019, 7:58 PM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Update branch date to match the 7.0.0 from 8.0.0 branch date.

Jan 15 2019, 1:18 PM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.
Jan 15 2019, 1:14 PM
erichkeane committed rC351212: Fix cpu-dispatch MV regression caused by r347812.
Fix cpu-dispatch MV regression caused by r347812
Jan 15 2019, 9:55 AM
erichkeane committed rL351212: Fix cpu-dispatch MV regression caused by r347812.
Fix cpu-dispatch MV regression caused by r347812
Jan 15 2019, 9:55 AM

Jan 11 2019

erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Fixed @lebedev.ri s comments.

Jan 11 2019, 10:48 AM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Added Policy guidance for the change process to CheckCompilerVersion.cmake.

Jan 11 2019, 10:33 AM
erichkeane committed rC350941: Fix a pair of Wfallthrough warnings in ScanfFormatString..
Fix a pair of Wfallthrough warnings in ScanfFormatString.
Jan 11 2019, 10:05 AM
erichkeane committed rL350941: Fix a pair of Wfallthrough warnings in ScanfFormatString..
Fix a pair of Wfallthrough warnings in ScanfFormatString.
Jan 11 2019, 10:05 AM

Jan 9 2019

erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Fixing my awful inattention to spelling in a few places, moving the AppleClang code to be more clear for future updates.

Jan 9 2019, 12:26 PM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Few more fixes suggested by @jfb . Also, added AppleClang to the check.

Jan 9 2019, 11:51 AM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.
Jan 9 2019, 10:46 AM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Change language to make this a 'guideline' instead of a hardline policy.

Jan 9 2019, 10:27 AM
erichkeane updated the diff for D47073: Document and Enforce new Host Compiler Policy.

Reworded a thing. Still needs direction feedback for 8.0.

Jan 9 2019, 6:25 AM
erichkeane added inline comments to D47073: Document and Enforce new Host Compiler Policy.
Jan 9 2019, 6:25 AM

Jan 8 2019

erichkeane committed rC350645: Fix opencl test broken on windows by r350643..
Fix opencl test broken on windows by r350643.
Jan 8 2019, 11:14 AM
erichkeane committed rL350645: Fix opencl test broken on windows by r350643..
Fix opencl test broken on windows by r350643.
Jan 8 2019, 11:14 AM
erichkeane committed rL350643: Limit COFF 'common' emission to <=32 alignment types..
Limit COFF 'common' emission to <=32 alignment types.
Jan 8 2019, 10:48 AM
erichkeane committed rC350643: Limit COFF 'common' emission to <=32 alignment types..
Limit COFF 'common' emission to <=32 alignment types.
Jan 8 2019, 10:48 AM
erichkeane closed D56391: Limit COFF 'common' emission to <=32 alignment types..
Jan 8 2019, 10:48 AM
erichkeane added a comment to D56407: Implement the TreeStructure interface through the TextNodeDumper.

I think this is a good approach to solving the problem, so let's go this route.

Jan 8 2019, 7:20 AM
erichkeane updated the diff for D56391: Limit COFF 'common' emission to <=32 alignment types..
Jan 8 2019, 6:16 AM

Jan 7 2019

erichkeane created D56391: Limit COFF 'common' emission to <=32 alignment types..
Jan 7 2019, 7:45 AM

Jan 4 2019

erichkeane added a comment to D47073: Document and Enforce new Host Compiler Policy.

Bumping again!

Jan 4 2019, 9:19 AM
erichkeane committed rC350398: Prevent unreachable when checking invalid multiversion decls..
Prevent unreachable when checking invalid multiversion decls.
Jan 4 2019, 7:31 AM
erichkeane committed rL350398: Prevent unreachable when checking invalid multiversion decls..
Prevent unreachable when checking invalid multiversion decls.
Jan 4 2019, 7:31 AM

Dec 18 2018

erichkeane committed rC349506: [NFC] Fix usage of Builder.insert(new Bitcast...)in CodeGenFunction.
[NFC] Fix usage of Builder.insert(new Bitcast...)in CodeGenFunction
Dec 18 2018, 8:27 AM
erichkeane committed rL349506: [NFC] Fix usage of Builder.insert(new Bitcast...)in CodeGenFunction.
[NFC] Fix usage of Builder.insert(new Bitcast...)in CodeGenFunction
Dec 18 2018, 8:26 AM

Dec 17 2018

erichkeane created D55778: Add /favor:<whatever> to CL-Compat's ignore list..
Dec 17 2018, 11:00 AM

Dec 14 2018

erichkeane committed rC349209: Add AddressSpace mangling to MS mode.
Add AddressSpace mangling to MS mode
Dec 14 2018, 3:20 PM
erichkeane committed rL349209: Add AddressSpace mangling to MS mode.
Add AddressSpace mangling to MS mode
Dec 14 2018, 3:20 PM
erichkeane closed D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 3:20 PM
erichkeane added inline comments to D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 3:15 PM
erichkeane committed rC349206: Revert "Add extension to always default-initialize nullptr_t.".
Revert "Add extension to always default-initialize nullptr_t."
Dec 14 2018, 2:45 PM
erichkeane committed rL349206: Revert "Add extension to always default-initialize nullptr_t.".
Revert "Add extension to always default-initialize nullptr_t."
Dec 14 2018, 2:45 PM
erichkeane added a comment to D53713: Add extension to always default-initialize nullptr_t..

As stated in CFE commits (in response to Richard's comments):

Dec 14 2018, 2:42 PM
erichkeane committed rL349201: Add extension to always default-initialize nullptr_t..
Add extension to always default-initialize nullptr_t.
Dec 14 2018, 2:26 PM
erichkeane committed rC349201: Add extension to always default-initialize nullptr_t..
Add extension to always default-initialize nullptr_t.
Dec 14 2018, 2:26 PM
erichkeane closed D53713: Add extension to always default-initialize nullptr_t..
Dec 14 2018, 2:25 PM
erichkeane updated the diff for D55715: Add AddressSpace mangling to MS mode.

Should catch me up on all comments except @zturner's llvm-undname feature request :)

Dec 14 2018, 2:09 PM
erichkeane added inline comments to D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 1:57 PM
erichkeane added inline comments to D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 11:36 AM
erichkeane added inline comments to D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 11:31 AM
erichkeane created D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 11:11 AM
erichkeane added a comment to D47073: Document and Enforce new Host Compiler Policy.

@jfb encouraged me to ping this, whats the current status/thoughts on this?

Dec 14 2018, 9:49 AM

Dec 13 2018

erichkeane added a comment to D53713: Add extension to always default-initialize nullptr_t..

Is there any feedback here? Am I just completely incorrect in how I tried to fix this?

Dec 13 2018, 11:44 AM

Dec 12 2018

erichkeane committed rC348969: Teach __builtin_unpredictable to work through implicit casts..
Teach __builtin_unpredictable to work through implicit casts.
Dec 12 2018, 12:34 PM
erichkeane committed rL348969: Teach __builtin_unpredictable to work through implicit casts..
Teach __builtin_unpredictable to work through implicit casts.
Dec 12 2018, 12:34 PM
erichkeane committed rL348950: Change CallGraph print to show the fully qualified name.
Change CallGraph print to show the fully qualified name
Dec 12 2018, 10:14 AM
erichkeane committed rC348950: Change CallGraph print to show the fully qualified name.
Change CallGraph print to show the fully qualified name
Dec 12 2018, 10:14 AM
erichkeane committed rL348942: Make clang::CallGraph look into template instantiations.
Make clang::CallGraph look into template instantiations
Dec 12 2018, 9:26 AM
erichkeane committed rC348942: Make clang::CallGraph look into template instantiations.
Make clang::CallGraph look into template instantiations
Dec 12 2018, 9:26 AM
erichkeane committed rL348932: Remove TODO leftover from my devleopment branch.
Remove TODO leftover from my devleopment branch
Dec 12 2018, 7:16 AM
erichkeane committed rC348932: Remove TODO leftover from my devleopment branch.
Remove TODO leftover from my devleopment branch
Dec 12 2018, 7:16 AM
erichkeane updated the diff for D55527: Normalize GlobalDecls when used with CPUDispatch.

As mentioned, the @rsmith comments that I thought were doable without feedback.

Dec 12 2018, 7:13 AM
erichkeane added a comment to D55527: Normalize GlobalDecls when used with CPUDispatch.

I got a couple of @rsmith's requests done. Most importantly I suspect is MultiVersionFuncs, though NotForDefinition vs ForDefinition is perhpas something you'll find important.

Dec 12 2018, 7:13 AM

Dec 11 2018

erichkeane added inline comments to D55527: Normalize GlobalDecls when used with CPUDispatch.
Dec 11 2018, 2:28 PM
erichkeane committed rC348899: Replace Const-Member checking with non-recursive version..
Replace Const-Member checking with non-recursive version.
Dec 11 2018, 1:58 PM
erichkeane committed rL348899: Replace Const-Member checking with non-recursive version..
Replace Const-Member checking with non-recursive version.
Dec 11 2018, 1:58 PM
erichkeane updated the diff for D55527: Normalize GlobalDecls when used with CPUDispatch.

Thanks @aaron.ballman . This should fix all of your complaints.

Dec 11 2018, 1:16 PM

Dec 10 2018

erichkeane created D55527: Normalize GlobalDecls when used with CPUDispatch.
Dec 10 2018, 11:53 AM