gchatelet retitled D53488: [clang-tidy] Improving narrowing conversions from [clang-tidy] Catching narrowing from double to float. to [clang-tidy] Improving narrowing conversions.

Nov 8 2018, 8:37 AM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

Is this good enough to go? @JonasToth

Nov 8 2018, 8:29 AM · Restricted Project

gchatelet accepted D54252: [llvm-exegesis][NFC] Add missing header guard + cosmetics..

Nov 8 2018, 2:59 AM

Nov 7 2018

gchatelet accepted D54151: [llvm-exegesis] Increasing wrapping limit..

Nov 7 2018, 1:28 AM

gchatelet added a comment to D54185: [PowerPC][llvm-exegesis] Add a PowerPC target.

Nice ! Thx for the patch !

Nov 7 2018, 12:54 AM

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Addressing comments

Nov 7 2018, 12:50 AM · Restricted Project

gchatelet added inline comments to D53488: [clang-tidy] Improving narrowing conversions.

Nov 7 2018, 12:50 AM · Restricted Project

Nov 6 2018

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Addressing comments and enhancing diagnostics

Nov 6 2018, 9:21 AM · Restricted Project

gchatelet accepted D54144: [llvm-exegesis] Correclty handle all X86 memory encoding formats..

Nov 6 2018, 9:12 AM

gchatelet added a comment to D54151: [llvm-exegesis] Increasing wrapping limit..

Did you run the test suite? We may have wrapping lines in the tests.

Nov 6 2018, 9:11 AM

gchatelet accepted D54147: [llvm-exegesis] Ignore X86 pseudo instructions..

Nov 6 2018, 5:59 AM

gchatelet added inline comments to D54147: [llvm-exegesis] Ignore X86 pseudo instructions..

Nov 6 2018, 5:01 AM

gchatelet added inline comments to D54144: [llvm-exegesis] Correclty handle all X86 memory encoding formats..

Nov 6 2018, 4:58 AM

Nov 5 2018

gchatelet added inline comments to D53488: [clang-tidy] Improving narrowing conversions.

Nov 5 2018, 2:33 AM · Restricted Project

Oct 30 2018

gchatelet added inline comments to D53488: [clang-tidy] Improving narrowing conversions.

Oct 30 2018, 3:26 PM · Restricted Project

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Address comments

Oct 30 2018, 3:24 PM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

Here are 15 random ones from llvm synced at 8f9fb8bab2e9b5b27fe40d700d2abe967b99fbb5.

lib/Target/ARM/AsmParser/ARMAsmParser.cpp:3802:31: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
tools/dsymutil/MachOUtils.cpp:330:10: warning: narrowing conversion from 'unsigned long' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/X86/X86ISelDAGToDAG.cpp:1237:14: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
tools/llvm-objdump/MachODump.cpp:7998:12: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/CodeGen/MachinePipeliner.cpp:3268:11: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/CodeGen/MIRParser/MIRParser.cpp:823:41: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Analysis/MemoryBuiltins.cpp:610:59: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Analysis/ValueTracking.cpp:2291:21: warning: narrowing conversion from 'unsigned long' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/ARM/ARMLoadStoreOptimizer.cpp:1470:43: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:2300:14: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/X86/X86TargetTransformInfo.cpp:2590:27: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/ARM/ARMFrameLowering.cpp:1770:27: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/ARM/ARMConstantIslandPass.cpp:514:22: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Transforms/Vectorize/LoopVectorize.cpp:5500:13: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]
lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1199:54: warning: narrowing conversion from 'int' to 'unsigned int' [cppcoreguidelines-narrowing-conversions]

Oct 30 2018, 9:16 AM · Restricted Project

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Remove leftover

Oct 30 2018, 9:16 AM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

So I ran llvm/tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py
The bare version produces 65664 unique findings.
The version restricted to cppcoreguidelines-narrowing-conversions produces 4323 unique findings. That's about +7% of findings.
I can post some random ones is you're interested.

Oct 30 2018, 8:07 AM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

So I've updated the code to add more narrowing conversions.
It now tests constant expressions against receiving type, do not warn about implicit bool casting, handles ternary operator with constant expressions.

Oct 30 2018, 7:28 AM · Restricted Project

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Add more checks, disable bool implicit casts warning, enable ternary operator warnings.

Oct 30 2018, 7:23 AM · Restricted Project

Oct 26 2018

gchatelet accepted D53766: [llvm-exegesis] Fix SNB counter definition and handling..

Oct 26 2018, 8:49 AM

Oct 25 2018

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

In D53488#1275786, @hokein wrote:

In D53488#1275750, @gchatelet wrote:

In D53488#1274205, @JonasToth wrote:

Did you run this code over a real-world code-base and did you find new stuff and/or false positives or the like?

Yes I did run it over our code base. I didn't find false positive but 98% of the warnings are from large generated lookup table initializers, e.g. const static float kTable[] = {0.0, 2.0, ... };
Since every number in the list triggers the warning, it accounts for most of them.

I scrutinized a few hundreds of the rest: none were actual bugs (although it's hard to tell sometimes), most are legit like float value = 0.0; but I also found some oddities from generated headers.

To me the warnings are useful and if it were my code I'd be willing to fix them. That said, I'd totally understand that many people would find them useless or annoying.
What do you think? Shall we still commit this as is?

It would be nice to know how many new findings does this patch introduce (number of findings before the patch vs after). If it is not too much, it is fine the commit as it is.
I'd suggest to run the check on llvm code repository (using clang-tidy/tool/run-clang-tidy.py, and only enable cppcoreguidelines-narrowing-conversions).

Oct 25 2018, 8:13 AM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

In D53488#1274205, @JonasToth wrote:

Did you run this code over a real-world code-base and did you find new stuff and/or false positives or the like?

Oct 25 2018, 7:42 AM · Restricted Project

Oct 24 2018

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Join the two parts of the ReleaseNotes update

Oct 24 2018, 7:00 AM · Restricted Project

gchatelet added a comment to D53488: [clang-tidy] Improving narrowing conversions.

In D53488#1273986, @JonasToth wrote:

Please add a short notice in the release notes for this change.

Oct 24 2018, 5:43 AM · Restricted Project

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Add documentation to ReleaseNotes

Oct 24 2018, 5:08 AM · Restricted Project

gchatelet committed rL345130: [llvm-exegesis] Implements a cache of Instruction objects..

[llvm-exegesis] Implements a cache of Instruction objects.

Oct 24 2018, 4:57 AM

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Update documentation, makes returning code path more explicit.

Oct 24 2018, 4:09 AM · Restricted Project

gchatelet added inline comments to D53488: [clang-tidy] Improving narrowing conversions.

Oct 24 2018, 4:09 AM · Restricted Project

gchatelet added inline comments to D53488: [clang-tidy] Improving narrowing conversions.

Oct 24 2018, 3:01 AM · Restricted Project

Oct 23 2018

gchatelet updated the diff for D53488: [clang-tidy] Improving narrowing conversions.

Addressing comments

Oct 23 2018, 4:43 AM · Restricted Project

Oct 22 2018

gchatelet created D53502: [llvm-exegesis] ExecutionMode is computed upfront for Uops as well.

Oct 22 2018, 8:13 AM

gchatelet committed rL344907: [llvm-exegesis] Crash when assembling invalid Operand.

[llvm-exegesis] Crash when assembling invalid Operand

Oct 22 2018, 8:08 AM

gchatelet committed rL344906: [llvm-exegesis] Mark x86 segment register instructions as unsupported..

[llvm-exegesis] Mark x86 segment register instructions as unsupported.

Oct 22 2018, 7:58 AM

gchatelet closed D53499: [llvm-exegesis] Mark x86 segment register instructions as unsupported..

Oct 22 2018, 7:57 AM

gchatelet created D53499: [llvm-exegesis] Mark x86 segment register instructions as unsupported..

Oct 22 2018, 7:56 AM

gchatelet committed rL344905: [llvm-exegesis] Reject x86 instructions that use non uniform memory accesses.

[llvm-exegesis] Reject x86 instructions that use non uniform memory accesses

Oct 22 2018, 7:48 AM

gchatelet closed D53438: [llvm-exegesis] Reject x86 instructions that use non uniform memory accesses.

Oct 22 2018, 7:48 AM

gchatelet updated the diff for D53438: [llvm-exegesis] Reject x86 instructions that use non uniform memory accesses.

Adding missing comment

Oct 22 2018, 2:55 AM

gchatelet updated the diff for D53438: [llvm-exegesis] Reject x86 instructions that use non uniform memory accesses.

Addressing comments

Oct 22 2018, 2:51 AM

gchatelet created D53488: [clang-tidy] Improving narrowing conversions.

Oct 22 2018, 2:20 AM · Restricted Project

gchatelet accepted D53455: [llvm-exegesis] Move namespace exegesis inside llvm::.

Nice ! Thx a lot :)

Oct 22 2018, 1:46 AM

Oct 19 2018

gchatelet created D53438: [llvm-exegesis] Reject x86 instructions that use non uniform memory accesses.

Oct 19 2018, 8:17 AM

gchatelet accepted D53430: [llvm-exegesis] Mark second-form X87 instructions as unsupported..

Oct 19 2018, 5:11 AM

gchatelet accepted D53429: [llvm-exegesis] Re-enable liveliness tracker..

Oct 19 2018, 3:59 AM

gchatelet accepted D53423: [llvm-exegesis] X87 RFP setup code..

Oct 19 2018, 2:50 AM

Oct 18 2018

gchatelet committed rL344731: [llvm-exegesis] Fix off by one error.

[llvm-exegesis] Fix off by one error

Oct 18 2018, 1:23 AM

Oct 17 2018

gchatelet accepted D53371: [llvm-exegesis] Allow measuring several instructions in a single run..

Oct 17 2018, 7:32 AM

gchatelet added inline comments to D53371: [llvm-exegesis] Allow measuring several instructions in a single run..

Oct 17 2018, 7:18 AM

gchatelet added a comment to D53371: [llvm-exegesis] Allow measuring several instructions in a single run..

In D53371#1267732, @lebedev.ri wrote:

How do you plan to deal with opcodes that crash (assertion failure) the llvm as a whole?
I.e. not when the snippet crashes, but the generation of the snippet itself crashes.
I *believe* there are some cases.

Oct 17 2018, 6:46 AM

gchatelet committed rL344692: Fix uninitialized variable.

Fix uninitialized variable

Oct 17 2018, 5:29 AM

gchatelet committed rL344690: BuildBot fix, compiler complains about array decay to pointer.

BuildBot fix, compiler complains about array decay to pointer

Oct 17 2018, 5:11 AM

gchatelet committed rL344689: [llvm-exegeis] Computing Latency configuration upfront so we can generate many….

[llvm-exegeis] Computing Latency configuration upfront so we can generate many…

Oct 17 2018, 4:40 AM

gchatelet closed D53320: [llvm-exegeis] Computing Latency configuration upfront so we can generate many CodeTemplates at once..

Oct 17 2018, 4:39 AM