rs1729 / RS

radiosonde decoding
GNU General Public License v3.0
170 stars 56 forks source link

demod_mod: simplify Makefile #50

Closed pinkavaj closed 1 year ago

rs1729 commented 1 year ago

Two issues here: 1) lms6mod is deprecated (not maintained anymore), lms6Xmod does both. As I said, there are some older version here and there... 2) I want -Ofast for demod_mod.c, speeds up calculations with floats a lot, FFT and lowpass. Possible inaccuracies are negligible. (Same for imet4iq.c, mk2a1680mod.c, dft_detect.c.)

pinkavaj commented 1 year ago
  1. lms6mod removed
  2. fixed for demod_mod.c, I did not touched the others (yet).
rs1729 commented 1 year ago

gcc -O3 -Ofast -c -o demod_mod.o demod_mod.c is as good as just -Ofast, i.e. -Ofastwon't be discarded? Just asking, seems to perform the same, though.

One more thing, the old *.o-files are not cleaned, my fault, didn't have this before (and got a segfault now... ;). Is there a simple way to delete also the o-files, other than rm *.o?

rs1729 commented 1 year ago

ok, I see, I didn't clean rs92mod.o, rs41mod.o etc., because they were not produced before. -O3 -Ofast should be ok, but could you include cleaning all the produced o-files?

Perhaps rm -f *.owill do, also for demod_mod.o, bch_ecc_mod.o.

pinkavaj commented 1 year ago

If multiple -Oxxx are passed, the only the last one is the actually used, so this should be ok.

Added clean-up for all the *.o, thx for spotting that!

rs1729 commented 1 year ago

Thanks!