On 06/13/17 17:44, Joe Savage wrote: >>>>> Joe, >>>>> >>>>> can you please make it work with clang? I sent a patch to ml before. It >>>>> might still apply, so you can review it. >>>>> https://travis-ci.org/muvarov/odp/jobs/223572921 >>>>> >>>>> the goal is to find good combination of -mcx16 and -latomic flags. And >>>>> we need to test that it still works on arm due to there is no mcx16 flag. >>>>> >>>>> Maxim. >>>> >>>> Hey Maxim, >>>> >>>> As we discussed previously, from my end the example should work perfectly >>>> with clang, it is just a matter of passing the right flags to the compiler >>>> through the project infrastructure. As such, you should be in a much better >>>> position to evaluate this than I. Looking at the descriptions of the >>>> patches >>>> alone, however, it looks like your patch combined with Brian Brooks' >>>> "configure.ac: fix native Clang build on ARMv8" should resolve any >>>> compilation issues here. >>>> >>>> Joe >>>> >>> >>> Ok, than if you want I can update my patch and send together with yours >>> as patchset. >>> >>> Maxim. >> >> Sure -- if that's how you'd like the patches to be grouped, go for it. > > Any progress on this, Maxim? >
Not yet. I just came from vacation :) But looks like it has to cleanly apply and build after merging "rework odp link with other libraries" serried with changes to cflags. Maxim.
