Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix string literal conversion warnings in env #7186

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

dylanjtuttle
Copy link
Contributor

@dylanjtuttle dylanjtuttle commented Nov 16, 2023

Work towards fixing AIX warnings about assigning string literals to non-const char pointers by adding 'const' qualifiers to some string variables and parameters (or worst case scenario, casting to (char *)) in env.

This PR contributes to (but does not close) eclipse-openj9/openj9#14859

This PR depends on:

and must be merged in coordination with eclipse-openj9/openj9#18467

Copy link
Contributor

@hzongaro hzongaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Please squash the two commits.

compiler/optimizer/Inliner.cpp Outdated Show resolved Hide resolved
compiler/optimizer/LoopVersioner.cpp Outdated Show resolved Hide resolved
@hzongaro
Copy link
Contributor

Jenkins build all

@dylanjtuttle
Copy link
Contributor Author

dylanjtuttle commented Nov 30, 2023

Build failures due to a link error:

16:06:07  /usr/bin/ld: fvtest/compilertest/libtestcompiler.a(OMRCodeGenPhase.cpp.o): in function `OMR::CodeGenPhase::performProcessRelocationsPhase(TR::CodeGenerator*, TR::CodeGenPhase*)':
16:06:07  OMRCodeGenPhase.cpp:(.text+0x88c): undefined reference to `setDllSlip(char*, char*, char*, TR::Compilation*)'
16:06:07  /usr/bin/ld: OMRCodeGenPhase.cpp:(.text+0x8dd): undefined reference to `setDllSlip(char*, char*, char*, TR::Compilation*)'

I'm not sure why it would be a link error instead of a regular compiler error, but maybe it's because I updated the parameter types of setDllSlip in FEBase.cpp, but then not anywhere else for some reason. I can fix that!

@hzongaro
Copy link
Contributor

hzongaro commented Dec 1, 2023

Build failures due to a link error:

Sorry I missed that! I guess you'll need to update any downstream implementation in OpenJ9 pull request eclipse-openj9/openj9#18467 as well.

Fix string literal conversion warnings in compiler/env

Signed-off-by: Dylan Tuttle <jdylantuttle@gmail.com>
Copy link
Contributor

@hzongaro hzongaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks!

@hzongaro
Copy link
Contributor

hzongaro commented Dec 4, 2023

Jenkins build all

@hzongaro
Copy link
Contributor

hzongaro commented Dec 4, 2023

There's an odd failure in eclipse.omr (x86-64 Linux):

42: 
41/50 Test #42: threadRealtimeTest ..................***Exception: SegFault  0.12 sec

I don't think it's related to this pull request, and I found at least one previous reference in pull request 6907 to that failure occurring. Re-running.

@hzongaro
Copy link
Contributor

hzongaro commented Dec 4, 2023

The x86-64 macOS failures appear to be due to issue #6516:

34: /Users/runner/work/1/s/fvtest/porttest/omrsockTest.cpp:1209: Failure
34:       Expected: rc
34:       Which is: 1
34: To be equal to: 2
34: [  FAILED  ] PortSockTest.poll_functionality_basic (1002 ms)
34: /Users/runner/work/1/s/fvtest/porttest/omrsockTest.cpp:51: Failure
34:       Expected: privateOmrPortLibrary->sock_bind(privateOmrPortLibrary, *serverSocket, serverSockAddr)
34:       Which is: -506
34: To be equal to: 0
34: /Users/runner/work/1/s/fvtest/porttest/omrsockTest.cpp:1259: Failure
34:       Expected: privateOmrPortLibrary->sock_accept(privateOmrPortLibrary, serverSocket, &connectedServerSockAddr, &connectedServerSocket)
34:       Which is: -20
34: To be equal to: 0
34: [  FAILED  ] PortSockTest.poll_functionality_many_sockets (1 ms)
34: [----------] 19 tests from PortSockTest (1008 ms total)
34: 
34: [==========] 246 tests from 21 test cases ran. (203845 ms total)
34: [  PASSED  ] 244 tests.
34: [  FAILED  ] 2 tests, listed below:
34: [  FAILED  ] PortSockTest.poll_functionality_basic
34: [  FAILED  ] PortSockTest.poll_functionality_many_sockets

@hzongaro
Copy link
Contributor

hzongaro commented Dec 8, 2023

Performing a coordinated merge with eclipse-openj9/openj9#18467

@hzongaro hzongaro merged commit 137778a into eclipse-omr:master Dec 8, 2023
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants