-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Conversation
// Division equivalence. | ||
|
||
// Built-in type: | ||
volatile auto d = int64_min; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this volatile
magic for ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing important any more. I wanted to check if runtime results are the same as compile-time results so I put volatile
to prevent the compiler to compute it up front. I'm not sure it works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, except for the not needed volatile
and maybe confusing name for utf
alias
I will change both. The |
8ad37e1
to
f74dbca
Compare
Codecov Report
@@ Coverage Diff @@
## develop #4064 +/- ##
===========================================
+ Coverage 65.74% 65.77% +0.02%
===========================================
Files 308 307 -1
Lines 22778 22831 +53
===========================================
+ Hits 14976 15017 +41
- Misses 7802 7814 +12
|
No description provided.