While trying to use the recently added OMR_PRId64 macro in the tril tests added in #3637, I noticed that my IDE was saying that the macro was expanding to lld on an x86-64 Linux, which is incorrect: on this system, int64_t is defined as long int not long long int, and so the correct format specifier is ld.
It seems that omrformatconsts.h checks for the presence of the PRId64 macro before using it (with the lld format specifier as a fallback if it doesn't), but does not include inttypes.h which is where that macro would be defined.
aviansie-ben
All 6 comments
FYI @rwy0717
Leonardo2718
on 29 Mar 2019
Out of curiosity which IDE do you use?
fjeremic
on 29 Mar 2019
@fjeremic Visual Studio Code. It takes some serious work to get the include paths set up right, but it seems to deal with extensible classes well enough to work with.
aviansie-ben
on 29 Mar 2019
Thanks. I'm using Visual Studio proper myself, and yes IPATH's were tricky but it works. Hopefully IDE support improves once we've made the switch to CMake as some IDEs support CMake projects out of the box.
fjeremic
on 29 Mar 2019
rwy0717
on 2 Apr 2019
Fixed in #3709.
0xdaryl
on 5 Apr 2019
Related issues
fjeremic
路
3Comments
samolisov
路
5Comments
0dvictor
路
5Comments
Leonardo2718
路
6Comments
0xdaryl
路
3Comments