hubert.reinterpretcast (Hubert Tong)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 27 2014, 8:34 PM (221 w, 6 h)

Recent Activity

Wed, Nov 14

hubert.reinterpretcast accepted D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.

LGTM.

Wed, Nov 14, 7:17 PM

Tue, Nov 13

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Tue, Nov 13, 7:49 AM

Mon, Nov 12

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Mon, Nov 12, 7:51 PM

Thu, Nov 8

hubert.reinterpretcast added a comment to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.

some test points check the error report for ambiguous call because of too many implicit cast choices from ext_vector_type to vector type.

It appears the answer is to update these tests and remove the restriction on the type class.

Thu, Nov 8, 9:01 PM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Thu, Nov 8, 11:38 AM

Sun, Nov 4

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Sun, Nov 4, 9:44 AM

Thu, Nov 1

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Thu, Nov 1, 6:15 AM

Fri, Oct 26

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Fri, Oct 26, 10:37 AM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Fri, Oct 26, 10:21 AM

Thu, Oct 25

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Thu, Oct 25, 10:22 PM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Thu, Oct 25, 7:55 PM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Thu, Oct 25, 8:26 AM

Oct 19 2018

hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Oct 19 2018, 7:47 PM
hubert.reinterpretcast requested changes to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Oct 19 2018, 7:04 PM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Oct 19 2018, 2:00 PM
hubert.reinterpretcast added inline comments to D53417: [Clang][Sema]Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error.
Oct 19 2018, 6:51 AM

Aug 16 2018

hubert.reinterpretcast committed rL339969: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.
[libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback
Aug 16 2018, 4:57 PM
hubert.reinterpretcast committed rCXX339969: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.
[libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback
Aug 16 2018, 4:57 PM
hubert.reinterpretcast closed D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.
Aug 16 2018, 4:57 PM
hubert.reinterpretcast updated subscribers of D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.

I can commit sometime today; thanks.

Aug 16 2018, 10:32 AM

Aug 15 2018

hubert.reinterpretcast updated the diff for D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.

Address review comments by Marshall

Aug 15 2018, 3:01 PM
hubert.reinterpretcast added a comment to D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.

Is this test that's being added libc++ specific, or would it apply to other implementations as well?

The test can apply to other implementations as well (although I am not sure how the initializer_list include behaves under C++03 on other implementations). Is there some other place to put such tests?

It seems that test/std/numerics/rand/rand.eng/rand.eng.mers/ may be the place to add the test. I am looking into it.

Aug 15 2018, 2:40 PM
hubert.reinterpretcast added a comment to D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.

Is this test that's being added libc++ specific, or would it apply to other implementations as well?

The test can apply to other implementations as well (although I am not sure how the initializer_list include behaves under C++03 on other implementations). Is there some other place to put such tests?

Aug 15 2018, 9:11 AM

Aug 14 2018

hubert.reinterpretcast created D50736: [libc++] Use correct rand.eng.mers all-zeroes seed sequence fallback.
Aug 14 2018, 1:53 PM

Aug 1 2018

hubert.reinterpretcast added inline comments to D50055: Update the coding standard about NFC changes and whitespace.
Aug 1 2018, 2:29 PM

Mar 21 2018

hubert.reinterpretcast added inline comments to D41217: [Concepts] Concept Specialization Expressions.
Mar 21 2018, 3:51 PM

Mar 16 2018

hubert.reinterpretcast added inline comments to D41217: [Concepts] Concept Specialization Expressions.
Mar 16 2018, 7:35 PM

Jan 9 2018

hubert.reinterpretcast committed rL322114: Profiling tests: Endianess XFAIL for powerpc- (32-bit).
Profiling tests: Endianess XFAIL for powerpc- (32-bit)
Jan 9 2018, 12:10 PM

Jan 4 2018

hubert.reinterpretcast committed rC321830: Use POSIX argument syntax in test rewrite-includes-messages.c.
Use POSIX argument syntax in test rewrite-includes-messages.c
Jan 4 2018, 3:05 PM
hubert.reinterpretcast committed rL321830: Use POSIX argument syntax in test rewrite-includes-messages.c.
Use POSIX argument syntax in test rewrite-includes-messages.c
Jan 4 2018, 3:05 PM
hubert.reinterpretcast committed rL321828: Use backslash escape, replacing xargs -0 in test macro-multiline.c.
Use backslash escape, replacing xargs -0 in test macro-multiline.c
Jan 4 2018, 2:59 PM
hubert.reinterpretcast committed rC321828: Use backslash escape, replacing xargs -0 in test macro-multiline.c.
Use backslash escape, replacing xargs -0 in test macro-multiline.c
Jan 4 2018, 2:59 PM
hubert.reinterpretcast closed D41544: Use backslash escape, replacing xargs -0 in test macro-multiline.c.
Jan 4 2018, 2:59 PM

Jan 3 2018

hubert.reinterpretcast committed rC321778: Replace cp -a in various Clang tests.
Replace cp -a in various Clang tests
Jan 3 2018, 5:17 PM
hubert.reinterpretcast committed rL321778: Replace cp -a in various Clang tests.
Replace cp -a in various Clang tests
Jan 3 2018, 5:17 PM
hubert.reinterpretcast closed D41545: Replace cp -a in various Clang tests.
Jan 3 2018, 5:16 PM
hubert.reinterpretcast updated the summary of D41545: Replace cp -a in various Clang tests.
Jan 3 2018, 5:12 PM
hubert.reinterpretcast updated the diff for D41545: Replace cp -a in various Clang tests.

Use cp -R, which is sufficient instead of cp -RPp

Jan 3 2018, 5:10 PM

Jan 2 2018

hubert.reinterpretcast added a comment to D41545: Replace cp -a in various Clang tests.

I'll commit with cp -R tomorrow then; thanks.

Jan 2 2018, 6:45 PM

Dec 29 2017

hubert.reinterpretcast added a comment to D41545: Replace cp -a in various Clang tests.

Maybe cp -R is sufficient? cp -RPp was the just-to-be-safe "minimal change".

Dec 29 2017, 10:43 AM
hubert.reinterpretcast added a comment to D41544: Use backslash escape, replacing xargs -0 in test macro-multiline.c.

Ping

Dec 29 2017, 10:41 AM

Dec 22 2017

hubert.reinterpretcast created D41545: Replace cp -a in various Clang tests.
Dec 22 2017, 9:21 AM
hubert.reinterpretcast created D41544: Use backslash escape, replacing xargs -0 in test macro-multiline.c.
Dec 22 2017, 9:03 AM

Dec 6 2017

hubert.reinterpretcast committed rL319992: Remove old concepts parsing code.
Remove old concepts parsing code
Dec 6 2017, 4:35 PM
hubert.reinterpretcast committed rC319992: Remove old concepts parsing code.
Remove old concepts parsing code
Dec 6 2017, 4:35 PM
hubert.reinterpretcast closed D40380: Remove old concepts parsing code by committing rC319992: Remove old concepts parsing code.
Dec 6 2017, 4:34 PM

Dec 4 2017

hubert.reinterpretcast accepted D40673: Add _Float128 as alias to __float128 to enable compilations on Fedora27/glibc2-26.

Looks good to me.

Dec 4 2017, 1:04 PM · Restricted Project

Dec 1 2017

hubert.reinterpretcast added inline comments to D40673: Add _Float128 as alias to __float128 to enable compilations on Fedora27/glibc2-26.
Dec 1 2017, 2:41 PM · Restricted Project

Nov 30 2017

hubert.reinterpretcast added inline comments to D40673: Add _Float128 as alias to __float128 to enable compilations on Fedora27/glibc2-26.
Nov 30 2017, 2:26 PM · Restricted Project

Nov 25 2017

hubert.reinterpretcast added inline comments to D40381: Parse concept definition.
Nov 25 2017, 9:18 PM

Nov 24 2017

hubert.reinterpretcast added inline comments to D40381: Parse concept definition.
Nov 24 2017, 8:00 AM
hubert.reinterpretcast added inline comments to D40381: Parse concept definition.
Nov 24 2017, 6:33 AM

Jun 30 2017

hubert.reinterpretcast committed rL306905: Fix PR 33189: Clang assertion on template destructor declaration.
Fix PR 33189: Clang assertion on template destructor declaration
Jun 30 2017, 3:44 PM
hubert.reinterpretcast closed D33833: Fix PR 33189: Clang assertion on template destructor declaration.
Jun 30 2017, 3:44 PM
hubert.reinterpretcast updated the summary of D33833: Fix PR 33189: Clang assertion on template destructor declaration.
Jun 30 2017, 3:33 PM

Jun 23 2017

hubert.reinterpretcast added a comment to D33833: Fix PR 33189: Clang assertion on template destructor declaration.

Can we get this patch reviewed by any chance?

@kuang_he; it is customary to "ping". In this case, "Ping 2".

Jun 23 2017, 3:38 PM

Jun 2 2017

hubert.reinterpretcast added inline comments to D33833: Fix PR 33189: Clang assertion on template destructor declaration.
Jun 2 2017, 4:24 PM
hubert.reinterpretcast retitled D33833: Fix PR 33189: Clang assertion on template destructor declaration from Fix Clang assertion on template destructor declaration to Fix PR 33189: Clang assertion on template destructor declaration.
Jun 2 2017, 4:21 PM

May 19 2017

hubert.reinterpretcast committed rL303472: Fix valid-for-expr ellipses eaten as invalid decl.
Fix valid-for-expr ellipses eaten as invalid decl
May 19 2017, 5:35 PM
hubert.reinterpretcast closed D33339: Fix valid-for-expr ellipses eaten as invalid decl.
May 19 2017, 5:35 PM
hubert.reinterpretcast updated the summary of D33339: Fix valid-for-expr ellipses eaten as invalid decl.
May 19 2017, 3:17 PM
hubert.reinterpretcast updated the diff for D33339: Fix valid-for-expr ellipses eaten as invalid decl.

Remove stray ellipsis treatment entirely

May 19 2017, 3:14 PM
hubert.reinterpretcast added a comment to D33339: Fix valid-for-expr ellipses eaten as invalid decl.

The check-all target passes even if the ellipsis-after-declarator-id disambiguation as a declarator is removed entirely.

[...]

So, on the whole, the stray ellipsis treatment is both too complicated and not complicated enough.

I think if we want to keep it, the way to do that would be to carry on through the disambiguation process and treat it as a tiebreaker (that's what we do, for instance, if we see an undeclared identifier in a position where we're looking for a type). I'm not convinced that's worthwhile, especially since our existing testcases do not need this disambiguation rule, but perhaps we could remove the stray ellipsis treatment entirely for now and reconsider adding it back if we find poor diagnostics result from it later?

May 19 2017, 1:21 PM

May 18 2017

hubert.reinterpretcast added a comment to D33339: Fix valid-for-expr ellipses eaten as invalid decl.

Should I assume our "misplaced ellipsis" diagnostic requires that we disambiguate the ill-formed ellipsis-after-declarator-id as a declarator in some cases? If so, do we have tests for that somewhere?

Tests for the diagnostics are in clang/test/FixIt/fixit-cxx0x.cpp and clang/test/Parser/cxx11-templates.cpp.
The check-all target passes even if the ellipsis-after-declarator-id disambiguation as a declarator is removed entirely. The disambiguation process is not needed in many cases, and—when it is—can choose the declarative interpretation for other reasons.

May 18 2017, 9:01 PM
hubert.reinterpretcast added inline comments to D33339: Fix valid-for-expr ellipses eaten as invalid decl.
May 18 2017, 2:59 PM
hubert.reinterpretcast created D33339: Fix valid-for-expr ellipses eaten as invalid decl.
May 18 2017, 2:28 PM

Apr 30 2017

hubert.reinterpretcast added a comment to D32378: Insert invariant.group.barrier for pointers comparisons.

Has it been discussed whether this is something to be addressed in the optimizer as opposed to the front-end?

Apr 30 2017, 8:38 PM

Feb 9 2017

hubert.reinterpretcast added inline comments to D29748: [cxx1z-constexpr-lambda] Implement captures - thus completing implementation of constexpr lambdas..
Feb 9 2017, 7:25 PM · Restricted Project
hubert.reinterpretcast committed rL294697: [Concepts] Class template associated constraints.
[Concepts] Class template associated constraints
Feb 9 2017, 6:57 PM
hubert.reinterpretcast closed D25674: [Concepts] Class template associated constraints.
Feb 9 2017, 6:57 PM
hubert.reinterpretcast updated the diff for D25674: [Concepts] Class template associated constraints.

Address review comments; update to revision 294580

Feb 9 2017, 12:04 PM

Feb 8 2017

hubert.reinterpretcast added inline comments to D25674: [Concepts] Class template associated constraints.
Feb 8 2017, 4:34 PM
hubert.reinterpretcast added a comment to D25674: [Concepts] Class template associated constraints.

Ping 2?

Feb 8 2017, 2:47 PM

Feb 1 2017

hubert.reinterpretcast added a comment to D25674: [Concepts] Class template associated constraints.

Ping!

Feb 1 2017, 11:23 AM

Jan 25 2017

hubert.reinterpretcast added inline comments to D25674: [Concepts] Class template associated constraints.
Jan 25 2017, 7:25 AM
hubert.reinterpretcast updated the diff for D25674: [Concepts] Class template associated constraints.

Address review comments; update to revision 292996

Jan 25 2017, 7:10 AM

Dec 1 2016

hubert.reinterpretcast added inline comments to D26817: [APFloat] Implement PPCDoubleDouble add and subtract..
Dec 1 2016, 10:29 AM

Nov 21 2016

hubert.reinterpretcast committed rL287583: reassociate-deadinst.ll: avoid accidental match on path.
reassociate-deadinst.ll: avoid accidental match on path
Nov 21 2016, 2:02 PM

Nov 20 2016

hubert.reinterpretcast added inline comments to D26882: Refactor how FunctionDecl handles constexpr:.
Nov 20 2016, 5:13 PM

Oct 24 2016

hubert.reinterpretcast added a comment to D25674: [Concepts] Class template associated constraints.

Friendly ping...

Oct 24 2016, 11:26 AM

Oct 22 2016

hubert.reinterpretcast added inline comments to D25674: [Concepts] Class template associated constraints.
Oct 22 2016, 7:33 PM

Oct 17 2016

hubert.reinterpretcast retitled D25674: [Concepts] Class template associated constraints from to [Concepts] Class template associated constraints.
Oct 17 2016, 6:34 AM

Oct 11 2016

hubert.reinterpretcast added a comment to D25480: __builtin_fpclassify missing one int parameter.

__builtin_fpclassify takes five int arguments followed by one last argument that is of floating point type. Do you know if there is a way to specify the last one argument is a floating point rather than using '.'?

There is already code in clang/lib/Sema/SemaChecking.cpp to generate diag::err_typecheck_call_invalid_unary_fp.

Oct 11 2016, 12:21 PM
hubert.reinterpretcast added a comment to D25480: __builtin_fpclassify missing one int parameter.

A test should probably be added as well.

Oct 11 2016, 10:50 AM

Sep 28 2016

hubert.reinterpretcast added inline comments to D22745: Added -ftemplate-auto option to enable the template auto feature (P0127). I am the author of the proposal; I believe this to be a complete implementation. Currently, the feature is enabled through the use of the -ftemplate-auto option and is not....
Sep 28 2016, 7:18 PM

Aug 30 2016

hubert.reinterpretcast added a comment to D24011: [ConstantFold] Add a flag for ppc_fp128 constant folding, since APFloat doesn't support double-double semantic.

If there seems to be few applications are depending on this optimization, I wonder if we can turn off the constant folding for now, and fix APFloat later if someone really cares about it?

Just wanted to note that the front-end would still encounter the issue with APFloat, and I would be concerned if the (front-end) constant expression evaluation for ppc_fp128 is disabled entirely.

Aug 30 2016, 2:04 PM

Aug 7 2016

hubert.reinterpretcast added inline comments to D22996: [cxx1z-constexpr-lambda] Implement constant evaluation of non-capturing lambda expressions..
Aug 7 2016, 3:43 PM · Restricted Project
hubert.reinterpretcast committed rL277968: [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor.
[libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor
Aug 7 2016, 3:34 PM
hubert.reinterpretcast committed rL277967: Revert r277966. Forgot patch attribution..
Revert r277966. Forgot patch attribution.
Aug 7 2016, 3:31 PM
hubert.reinterpretcast committed rL277966: [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor.
[libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor
Aug 7 2016, 3:26 PM
hubert.reinterpretcast closed D22702: [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor.
Aug 7 2016, 3:26 PM
hubert.reinterpretcast retitled D22702: [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor from [libcxx]Missing default argument for flag_type parameter in one of std::basic_regex constructors to [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor.
Aug 7 2016, 2:46 PM
hubert.reinterpretcast accepted D22702: [libcxx] Add "flag" default arg: basic_regex ptr_size_flag ctor.

LGTM.

Aug 7 2016, 8:29 AM

Aug 3 2016

hubert.reinterpretcast committed rL277658: [Concepts] remove default argument for RequiresClause; NFC.
[Concepts] remove default argument for RequiresClause; NFC
Aug 3 2016, 3:15 PM

Aug 2 2016

hubert.reinterpretcast committed rL277526: [libcxx] basic_regex: add traits_type, string_type.
[libcxx] basic_regex: add traits_type, string_type
Aug 2 2016, 2:42 PM
hubert.reinterpretcast closed D22698: [libcxx] basic_regex: add traits_type, string_type .
Aug 2 2016, 2:42 PM
hubert.reinterpretcast retitled D22698: [libcxx] basic_regex: add traits_type, string_type from [libcxx] Missing member types 'traits_type' and 'string_type' in class basic_regex to [libcxx] basic_regex: add traits_type, string_type .
Aug 2 2016, 1:50 PM
hubert.reinterpretcast committed rL277495: [Concepts] Add TODO and requires-clause placeholder; NFC.
[Concepts] Add TODO and requires-clause placeholder; NFC
Aug 2 2016, 11:44 AM
hubert.reinterpretcast accepted D22698: [libcxx] basic_regex: add traits_type, string_type .

LGTM.

Aug 2 2016, 9:20 AM

Jul 30 2016

hubert.reinterpretcast committed rL277286: Reapply r276069 with workaround for MSVC 2013.
Reapply r276069 with workaround for MSVC 2013
Jul 30 2016, 3:41 PM