<div dir="ltr"><div dir="ltr">Simon,<div><br></div><div>See below...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 11:24 AM Simon Marchi <<a href="mailto:simark@simark.ca">simark@simark.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2020-03-23 12:56 p.m., Rocky Dunlap wrote:<br>
> Simon,<br>
> <br>
> Success!  This change worked and I was able to build and install with icc.  Thank you!<br>
<br>
Ok, thanks.  Out of curiosity, did run a "make check"?  That would validate that<br>
the Python bindings (and everything else) works properly.<br></blockquote><div><br></div><div>============================================================================<br>Testsuite summary for babeltrace2 2.1.0-rc1<br>============================================================================<br># TOTAL: 21225<br># PASS:  21225<br># SKIP:  0<br># XFAIL: 0<br># FAIL:  0<br># XPASS: 0<br># ERROR: 0<br>============================================================================<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> I still get a lot of warnings during the build (see below) which seems like configure should work these out ahead of time.  They are easy enough to ignore, but if support for intel is added, it would be nice to have these cleaned up.  For what it's worth, adding support for additional compilers in my experience has greatly improved the quality of library code - it tends to snuff out those bugs that are hard to find on a single compiler.  <br>
<br>
I agree, that's why we want to keep babeltrace building with clang (on top of having to<br>
support macOS, on which clang is the default compiler).<br>
<br>
We use the AX_APPEND_COMPILE_FLAGS autoconf macro [1] to determine which warning flags<br>
the current compiler supports.  This calls the compiler with each flag individually<br>
to see if the compiler supports that flag.  This requires that the compiler errors<br>
out when passed an invalid flag.  Currently, it looks like icc emits a warning, but still<br>
returns exit code 0 when passed an invalid flag, so we end up enabling all of them.  Try<br>
to check in config.log to see what happens currently.  For example here are the test<br>
results for two flags, one supported and one not supported by clang:<br>
<br>
4625 configure:17825: checking whether C compiler accepts -Winit-self<br>
4626 configure:17844: ccache clang -c -gdwarf-5 -g3 -O2 -fsanitize=address -Werror -Winit-self  conftest.c >&5<br>
4627 configure:17844: $? = 0<br>
4628 configure:17853: result: yes<br>
4629 configure:17869: : WARN_CFLAGS="$WARN_CFLAGS"<br>
4630 configure:17872: $? = 0<br>
4631 configure:17825: checking whether C compiler accepts -Wduplicated-cond<br>
4632 configure:17844: ccache clang -c -gdwarf-5 -g3 -O2 -fsanitize=address -Werror -Wduplicated-cond  conftest.c >&5<br>
4633 error: unknown warning option '-Wduplicated-cond'; did you mean '-Wduplicate-enum'? [-Werror,-Wunknown-warning-option]<br>
4634 configure:17844: $? = 1<br>
<br>
If you can come up with the right incantation to make that work well with icc, we<br>
can modify how we call the macro or modify the macro itself (sending a patch to<br>
the upstream, Autoconf Archive).<br>
<br>
[1] <a href="https://www.gnu.org/software/autoconf-archive/ax_append_compile_flags.html" rel="noreferrer" target="_blank">https://www.gnu.org/software/autoconf-archive/ax_append_compile_flags.html</a></blockquote><div><br></div><div>Okay, I'll try to find some time look into this.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
> If you decide to merge this in, a release patch 2.0.3 would certainly be helpful as well!<br>
<br>
Things have started to diverge a bit between master and the 2.0 branch, I'm not sure if we'll<br>
be able to backport that, we'll see.<br></blockquote><div><br></div><div>Sure, no problem.  I'm not sure what the overall release schedule is, so maybe it would be a 2.1 at some point soon?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Simon<br>
> <br>
<br>
</blockquote></div></div>