ompi icon indicating copy to clipboard operation
ompi copied to clipboard

Fix a bunch of compiler warnings in "make check"

Open jsquyres opened this issue 4 years ago • 5 comments

There were many compiler warnings, making it difficult to read if there were real issues that needed to be addressed.

Signed-off-by: Jeff Squyres [email protected]

jsquyres avatar Apr 24 '21 14:04 jsquyres

IBM CI had a bad weekend (sorry about that). We should be all good now. bot:ibm:pgi:retest bot:ibm:xl:retest

jjhursey avatar Apr 26 '21 15:04 jjhursey

@jsquyres Can you please rebase? This may be important.

gpaulsen avatar May 21 '21 18:05 gpaulsen

bot:retest /azp run bot:ibm:retest

gpaulsen avatar Jan 12 '22 17:01 gpaulsen

bot:ibm:retest

gpaulsen avatar Jan 12 '22 18:01 gpaulsen

Any reason to hold this PR any longer? (other than it need to be rebased)

jjhursey avatar Aug 04 '22 18:08 jjhursey

I rebased on tip of main(no code change).

gpaulsen avatar Sep 26 '22 18:09 gpaulsen

If that 3rd commit passes CI properly, you'll squash it appropriately, right? (minor pet peeve: make a commit on a PR and then make a later commit on the same PR to fix the earlier commit)

jsquyres avatar Sep 26 '22 20:09 jsquyres

mellanox CI failed that it couldn't find a definition for the non-sized versions of these, and I searched around and couldn't find those definitions either, so I created the 3rd commit. Why the other CIs didn't fail, I'm not sure. @bwbarrett thoughts?

gpaulsen avatar Sep 26 '22 21:09 gpaulsen

I git bisected opal/include/opal/sys/atomic_stdc.h and couldn't find a definition for opal_atomic_compare_exchange_strong anywhere in the past.

gpaulsen avatar Sep 26 '22 21:09 gpaulsen

bot:aws:retest

bwbarrett avatar Sep 28 '22 02:09 bwbarrett