Page MenuHomePhabricator

rtrieu (Richard Trieu)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 11 2012, 7:44 PM (427 w, 3 d)

Recent Activity

Aug 12 2020

rtrieu accepted D85778: More accurately compute the ranges of possible values for +, -, *, &, %..

LGTM

Aug 12 2020, 7:01 PM · Restricted Project

Aug 10 2020

rtrieu accepted D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators.

LGTM

Aug 10 2020, 7:01 PM · Restricted Project
rtrieu added inline comments to D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators.
Aug 10 2020, 6:02 PM · Restricted Project

Aug 6 2020

rtrieu added a comment to D85287: Extend -Wtautological-bitwise-compare "bitwise or with non-zero value" warnings.

Also, have you tried running warning over a codebase?

As I wrote: "At least building LibreOffice with this change caused no false positives." (Or maybe I misunderstand your question.)

Aug 6 2020, 7:10 PM · Restricted Project
rtrieu updated subscribers of D85287: Extend -Wtautological-bitwise-compare "bitwise or with non-zero value" warnings.

I looked back on the commits and while I did commit, I did it on the behalf of Anders Rönnholm, who did not have commit access at the time. I haven't seen activity from Anders in a while, but added to subscribers just in case.

Aug 6 2020, 4:46 AM · Restricted Project
rtrieu accepted D85256: Add -Wtautological-value-range-compare warning..

LGTM

Aug 6 2020, 4:17 AM · Restricted Project

Jun 19 2020

rtrieu committed rGd5f9c4a3d10d: [ODRHash] Remove use of 'whitelist'. (authored by rtrieu).
[ODRHash] Remove use of 'whitelist'.
Jun 19 2020, 7:00 PM

May 7 2020

rtrieu committed rG4ae537c2220f: Fix false positive with -Wnon-c-typedef-for-linkage (authored by rtrieu).
Fix false positive with -Wnon-c-typedef-for-linkage
May 7 2020, 8:05 PM
rtrieu closed D79548: Fix false positive with warning -Wnon-c-typedef-for-linkage.
May 7 2020, 8:04 PM · Restricted Project

May 6 2020

rtrieu created D79548: Fix false positive with warning -Wnon-c-typedef-for-linkage.
May 6 2020, 10:08 PM · Restricted Project

Apr 27 2020

rtrieu abandoned D60169: Proposed refactoring for lib/Target/X86 to remove layering issue.
Apr 27 2020, 5:48 PM

Mar 31 2020

rtrieu committed rGf93aed66f22f: Fix diagnostics where _Atomic can't be applied (authored by rtrieu).
Fix diagnostics where _Atomic can't be applied
Mar 31 2020, 5:40 PM

Jan 30 2020

rtrieu created D73762: [clang] New warning for for-loops where the iteration does not match the loop condition.
Jan 30 2020, 6:41 PM · Restricted Project

Jan 24 2020

rtrieu committed rG9f69157bf4a0: Fix header includes after 0697bcb66f1d82f2fd447e9d13b74d141c3ce085 (authored by rtrieu).
Fix header includes after 0697bcb66f1d82f2fd447e9d13b74d141c3ce085
Jan 24 2020, 6:34 PM

Jan 23 2020

rtrieu committed rGfe5f233a938f: Fix assert that doesn't check anything. (authored by rtrieu).
Fix assert that doesn't check anything.
Jan 23 2020, 7:16 PM
rtrieu added a comment to D73007: [Sema] Avoid Wrange-loop-analysis false positives.

I'm in favor of splitting the warning into subgroups, then deciding which ones should be in -Wall. We've done this with other warnings such as the conversion warnings and tautological compare warnings, with only a subset of them in -Wall.

Jan 23 2020, 5:00 PM · Restricted Project

Jan 21 2020

rtrieu added inline comments to D72552: [Concepts] Constraint Satisfaction Caching.
Jan 21 2020, 5:53 PM · Restricted Project

Jan 14 2020

rtrieu added a comment to D71734: [Modules] Handle tag types and complain about bad merges in C/Objective-C mode.

Heads up in case it affects you refactoring work:
While looking through this code, I found a bug I previously made. I fixed it with a small change, but that lies in the middle of your refactoring during FieldDecl handling. The change is here:

Jan 14 2020, 10:08 PM · Restricted Project
rtrieu committed rGa60e89272970: [ODRHash] Fix wrong error message with bitfields and mutable. (authored by rtrieu).
[ODRHash] Fix wrong error message with bitfields and mutable.
Jan 14 2020, 9:31 PM

Jan 10 2020

rtrieu created D72551: Warn when a string literal is used in a range-based for-loop.
Jan 10 2020, 6:46 PM
rtrieu added inline comments to D71734: [Modules] Handle tag types and complain about bad merges in C/Objective-C mode.
Jan 10 2020, 6:18 PM · Restricted Project

Jan 9 2020

rtrieu added inline comments to D71734: [Modules] Handle tag types and complain about bad merges in C/Objective-C mode.
Jan 9 2020, 6:53 PM · Restricted Project

Jan 8 2020

rtrieu added inline comments to D72212: [Sema] Improve -Wrange-loop-analysis warnings.
Jan 8 2020, 7:10 PM · Restricted Project

Dec 16 2019

rtrieu updated the diff for D71503: New warning on for-loops that never run because condition is false on the first iteration.
Dec 16 2019, 3:57 PM
rtrieu added a comment to D71503: New warning on for-loops that never run because condition is false on the first iteration.

Thank you for working on this!
This seems to be missing tests.

Dec 16 2019, 3:57 PM

Dec 13 2019

rtrieu created D71503: New warning on for-loops that never run because condition is false on the first iteration.
Dec 13 2019, 7:04 PM

Nov 15 2019

rtrieu added a comment to D69292: Proposal to add -Wtautological-compare to -Wall.

Sorry I'm late to the party; I've been traveling for 3+ weeks.
I would like to be reassured that the following code will not warn:

`
long foo = ...; // some calculation
if (foo < std::numeric_limits<int>::min() || foo > std::numeric_limits<int>::max()) .....

This is important for systems where sizeof(int) == sizeof(long)

Nov 15 2019, 4:09 PM · Restricted Project

Nov 12 2019

rtrieu committed rG9740f9f0b6e5: Add -Wtautological-compare to -Wall (authored by rtrieu).
Add -Wtautological-compare to -Wall
Nov 12 2019, 2:50 PM
rtrieu closed D69292: Proposal to add -Wtautological-compare to -Wall.
Nov 12 2019, 2:50 PM · Restricted Project

Nov 7 2019

rtrieu updated the diff for D69292: Proposal to add -Wtautological-compare to -Wall.

Add -Wall tests to check that certain warning groups are active with it and a test to check all warning groups that are turned on by -Wall.

Nov 7 2019, 9:23 PM · Restricted Project

Nov 6 2019

rtrieu added a comment to D68577: [LV] Apply sink-after & interleave-groups as VPlan transformations (NFC).

The recommited patch is hitting an assert. See message and reduced test case below:

Nov 6 2019, 8:23 PM · Restricted Project

Oct 25 2019

rtrieu added a comment to D69292: Proposal to add -Wtautological-compare to -Wall.

Abstractly this lgtm. Do you have any data on true / false positive rates?

Oct 25 2019, 5:42 PM · Restricted Project

Oct 21 2019

rtrieu added a comment to D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.

Mr Trieu, what do you think about adding some or all of the Wtautological-compare warnings to Wall

Oct 21 2019, 8:04 PM · Restricted Project
rtrieu created D69292: Proposal to add -Wtautological-compare to -Wall.
Oct 21 2019, 7:45 PM · Restricted Project

Oct 18 2019

rtrieu committed rG637af4cc3780: Add -Wbitwise-conditional-parentheses to warn on mixing '|' and '&' with "?:" (authored by rtrieu).
Add -Wbitwise-conditional-parentheses to warn on mixing '|' and '&' with "?:"
Oct 18 2019, 6:52 PM
rtrieu closed D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Oct 18 2019, 6:52 PM · Restricted Project
rtrieu committed rL375326: Add -Wbitwise-conditional-parentheses to warn on mixing '|' and '&' with "?:".
Add -Wbitwise-conditional-parentheses to warn on mixing '|' and '&' with "?:"
Oct 18 2019, 6:52 PM
rtrieu committed rG8b0d14a8f0cc: New tautological warning for bitwise-or with non-zero constant always true. (authored by rtrieu).
New tautological warning for bitwise-or with non-zero constant always true.
Oct 18 2019, 5:57 PM
rtrieu committed rL375318: New tautological warning for bitwise-or with non-zero constant always true..
New tautological warning for bitwise-or with non-zero constant always true.
Oct 18 2019, 5:57 PM
rtrieu closed D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Oct 18 2019, 5:57 PM · Restricted Project

Oct 10 2019

rtrieu added a comment to D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Oct 10 2019, 8:59 PM · Restricted Project

Oct 4 2019

rtrieu added inline comments to rL373614: [Diagnostics] Bitwise negation of a boolean expr always evaluates to true; warn….
Oct 4 2019, 5:07 PM

Oct 1 2019

rtrieu committed rGe3887253165c: Revert r368237 - Update fix-it hints for std::move warnings. (authored by rtrieu).
Revert r368237 - Update fix-it hints for std::move warnings.
Oct 1 2019, 7:33 PM
rtrieu committed rL373421: Revert r368237 - Update fix-it hints for std::move warnings..
Revert r368237 - Update fix-it hints for std::move warnings.
Oct 1 2019, 7:30 PM

Sep 20 2019

rtrieu committed rG77297f0761d2: Fix bad APInt compare. (authored by rtrieu).
Fix bad APInt compare.
Sep 20 2019, 9:20 PM
rtrieu committed rL372454: Fix bad APInt compare..
Fix bad APInt compare.
Sep 20 2019, 9:17 PM
rtrieu committed rG4c05de8c1d15: Merge and improve code that detects same value in comparisons. (authored by rtrieu).
Merge and improve code that detects same value in comparisons.
Sep 20 2019, 8:05 PM
rtrieu committed rL372453: Merge and improve code that detects same value in comparisons..
Merge and improve code that detects same value in comparisons.
Sep 20 2019, 8:04 PM
rtrieu closed D66045: Improve detection of same value in comparisons.
Sep 20 2019, 8:04 PM · Restricted Project
rtrieu committed rG6541c7988b83: Improve -Wtautological-overlap-compare (authored by rtrieu).
Improve -Wtautological-overlap-compare
Sep 20 2019, 7:37 PM
rtrieu committed rL372448: Improve -Wtautological-overlap-compare.
Improve -Wtautological-overlap-compare
Sep 20 2019, 7:37 PM
rtrieu closed D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions.
Sep 20 2019, 7:37 PM · Restricted Project

Sep 6 2019

rtrieu updated the diff for D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").

Add more test cases and split new warnings into a separate warning group, but still under -Wparentheses

Sep 6 2019, 6:55 PM · Restricted Project

Aug 28 2019

rtrieu committed rGe4a7f0182d40: Add requirement to test. (authored by rtrieu).
Add requirement to test.
Aug 28 2019, 5:47 PM
rtrieu committed rL370279: Add requirement to test..
Add requirement to test.
Aug 28 2019, 5:47 PM

Aug 27 2019

rtrieu committed rG1709f4847e0c: Revert r370105 - Update two x86 datalayouts for r370083, looks like racing… (authored by rtrieu).
Revert r370105 - Update two x86 datalayouts for r370083, looks like racing…
Aug 27 2019, 6:56 PM
rtrieu committed rL370147: Revert r370105 - Update two x86 datalayouts for r370083, looks like racing….
Revert r370105 - Update two x86 datalayouts for r370083, looks like racing…
Aug 27 2019, 6:55 PM

Aug 26 2019

rtrieu committed rG58e67b8aa38c: Revert r369927 - [DAGCombiner] Remove a bunch of redundant AddToWorklist calls. (authored by rtrieu).
Revert r369927 - [DAGCombiner] Remove a bunch of redundant AddToWorklist calls.
Aug 26 2019, 7:05 PM
rtrieu committed rL370006: Revert r369927 - [DAGCombiner] Remove a bunch of redundant AddToWorklist calls..
Revert r369927 - [DAGCombiner] Remove a bunch of redundant AddToWorklist calls.
Aug 26 2019, 7:04 PM
rtrieu added a comment to D66537: [DAGCombiner] Remove a bunch of redundant AddToWorklist calls..

I have reverted this in r370006.

Aug 26 2019, 7:04 PM · Restricted Project
rtrieu committed rGf837bb4a3443: Copy test data so tests don't traverse test directories. NFC (authored by rtrieu).
Copy test data so tests don't traverse test directories. NFC
Aug 26 2019, 3:42 PM
rtrieu committed rL369984: Copy test data so tests don't traverse test directories. NFC.
Copy test data so tests don't traverse test directories. NFC
Aug 26 2019, 3:41 PM

Aug 19 2019

rtrieu committed rG8f9e489a6658: Fix typo. "piont" => "point" (authored by rtrieu).
Fix typo. "piont" => "point"
Aug 19 2019, 5:34 PM
rtrieu committed rL369316: Fix typo. "piont" => "point".
Fix typo. "piont" => "point"
Aug 19 2019, 5:34 PM

Aug 13 2019

rtrieu added a comment to D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions.
In D66044#1626008, @NoQ wrote:

Looks great, thanks! I'd appreciate direct CFG tests for the part that changes the CFG (cf. test/Analysis/cfg.cpp), but i don't insist. Let's make sure @jfb is happy and commit :)

Aug 13 2019, 8:12 PM · Restricted Project
rtrieu updated the diff for D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions.
Aug 13 2019, 8:08 PM · Restricted Project
rtrieu added inline comments to D66045: Improve detection of same value in comparisons.
Aug 13 2019, 6:33 PM · Restricted Project
rtrieu updated the diff for D66045: Improve detection of same value in comparisons.
Aug 13 2019, 6:20 PM · Restricted Project
rtrieu updated the diff for D66048: Combined patch for new warning work.

Better handling of arrays in self compares.

Aug 13 2019, 4:40 PM
rtrieu added inline comments to D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Aug 13 2019, 4:09 PM · Restricted Project

Aug 12 2019

rtrieu updated the diff for D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").

Update comments to explain why bitwise-xor is treated as a logical operator.

Aug 12 2019, 7:38 PM · Restricted Project
rtrieu added inline comments to D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Aug 12 2019, 7:38 PM · Restricted Project
rtrieu added inline comments to D66045: Improve detection of same value in comparisons.
Aug 12 2019, 7:29 PM · Restricted Project
rtrieu updated the diff for D66045: Improve detection of same value in comparisons.

Check array accesses.

Aug 12 2019, 7:25 PM · Restricted Project
rtrieu added inline comments to D66045: Improve detection of same value in comparisons.
Aug 12 2019, 4:40 PM · Restricted Project
rtrieu added inline comments to D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Aug 12 2019, 4:23 PM · Restricted Project
rtrieu updated the diff for D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.

Create new function isIntOrEnumConstantZero to use instead of EvaluateAsInt

Aug 12 2019, 4:19 PM · Restricted Project
rtrieu updated the summary of D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Aug 12 2019, 3:29 PM · Restricted Project
rtrieu added inline comments to D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Aug 12 2019, 3:29 PM · Restricted Project
rtrieu added a comment to D66045: Improve detection of same value in comparisons.
In D66045#1624389, @jfb wrote:

Does this work for unions as well?

Aug 12 2019, 3:17 PM · Restricted Project
rtrieu added inline comments to D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Aug 12 2019, 2:52 PM · Restricted Project
rtrieu added inline comments to D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions.
Aug 12 2019, 2:40 PM · Restricted Project
rtrieu added inline comments to D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Aug 12 2019, 2:28 PM · Restricted Project

Aug 9 2019

rtrieu added a reviewer for D66045: Improve detection of same value in comparisons: cfe-commits.
Aug 9 2019, 7:04 PM · Restricted Project
rtrieu added a reviewer for D66046: Add new tautological compare warning for bitwise-or with a non-zero constant: cfe-commits.
Aug 9 2019, 7:04 PM · Restricted Project
rtrieu added a reviewer for D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions: cfe-commits.
Aug 9 2019, 7:04 PM · Restricted Project
rtrieu added a reviewer for D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:"): cfe-commits.
Aug 9 2019, 7:04 PM · Restricted Project
rtrieu created D66048: Combined patch for new warning work.
Aug 9 2019, 7:04 PM
rtrieu created D66045: Improve detection of same value in comparisons.
Aug 9 2019, 6:55 PM · Restricted Project
rtrieu created D66046: Add new tautological compare warning for bitwise-or with a non-zero constant.
Aug 9 2019, 6:55 PM · Restricted Project
rtrieu created D66043: Add to -Wparentheses case of bitwise-and ("&") and bitwise-or ("|") verses conditional operator ("?:").
Aug 9 2019, 6:55 PM · Restricted Project
rtrieu created D66044: Extend -Wtautological-overlap-compare to accept negative values and integer conversions.
Aug 9 2019, 6:55 PM · Restricted Project

Aug 7 2019

rtrieu committed rG07e6798baf9a: Inline diagnostic text into .td file. NFC. (authored by rtrieu).
Inline diagnostic text into .td file. NFC.
Aug 7 2019, 6:47 PM
rtrieu committed rL368244: Inline diagnostic text into .td file. NFC..
Inline diagnostic text into .td file. NFC.
Aug 7 2019, 6:47 PM
rtrieu committed rG155b8d02c363: Update fix-it hints for std::move warnings. (authored by rtrieu).
Update fix-it hints for std::move warnings.
Aug 7 2019, 5:17 PM
rtrieu committed rL368237: Update fix-it hints for std::move warnings..
Update fix-it hints for std::move warnings.
Aug 7 2019, 5:12 PM

Jul 30 2019

rtrieu committed rG33cdbff2a78a: Change '|' to '&' in conditional. (authored by rtrieu).
Change '|' to '&' in conditional.
Jul 30 2019, 9:42 PM
rtrieu committed rL367386: Change '|' to '&' in conditional..
Change '|' to '&' in conditional.
Jul 30 2019, 9:40 PM

Jul 22 2019

rtrieu committed rG3a52c3857feb: Inline function call into assert to fix unused variable warning. (authored by rtrieu).
Inline function call into assert to fix unused variable warning.
Jul 22 2019, 8:11 PM
rtrieu committed rL366774: Inline function call into assert to fix unused variable warning..
Inline function call into assert to fix unused variable warning.
Jul 22 2019, 8:11 PM