Skip to content

v4.1.x: Make sure MPIR_Breakpoint() is compiled without CFLAGS.#8428

Merged
jsquyres merged 1 commit intoopen-mpi:v4.1.xfrom
awlauria:fix_mpir_breakpoint_v4.1.x
Jan 29, 2021
Merged

v4.1.x: Make sure MPIR_Breakpoint() is compiled without CFLAGS.#8428
jsquyres merged 1 commit intoopen-mpi:v4.1.xfrom
awlauria:fix_mpir_breakpoint_v4.1.x

Conversation

@awlauria
Copy link
Contributor

In optimized builds, CFLAGS contains various optimizations such as -O3,
and is propogated by automake to all files. To work-around this,
isolate MPIR_Breakpoint() and other MPIR_* symbols into its own library
built with debugger specific CFLAGS.

To prevent CFLAGS from being polluted elsewhere in the make tree, build
this in its own tiny stand-alone makefile.

Fixes #7757

Signed-off-by: Austen Lauria awlauria@us.ibm.com
(cherry picked from commit 6d82003)

In optimized builds, CFLAGS contains various optimizations such as -O3,
and is propogated by automake to all files. To work-around this,
isolate MPIR_Breakpoint() and other MPIR_* symbols into its own library
built with debugger specific CFLAGS.

To prevent CFLAGS from being polluted elsewhere in the make tree, build
this in its own tiny stand-alone makefile.

Fixes open-mpi#7757

Signed-off-by: Austen Lauria <awlauria@us.ibm.com>
(cherry picked from commit 6d82003)
@awlauria awlauria requested a review from jjhursey January 28, 2021 15:58
@awlauria awlauria changed the title Make sure MPIR_Breakpoint() is compiled without CFLAGS. v4.1.x: Make sure MPIR_Breakpoint() is compiled without CFLAGS. Jan 28, 2021
@jjhursey
Copy link
Member

bot:ompi:retest

@jjhursey jjhursey added this to the v4.1.1 milestone Jan 28, 2021
@jsquyres jsquyres merged commit 08b1f2d into open-mpi:v4.1.x Jan 29, 2021
@awlauria awlauria deleted the fix_mpir_breakpoint_v4.1.x branch January 29, 2021 20:43
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.

3 participants