There is a simple naming conflict when -DPIC is in the CFLAGS, breaking build of pic.cpp, since there is a local class with name PIC.
Of course I prompty see respectable programmable interrupt controllers complain that they existed before this position-independent code fad, but I still hope they'll understand that they have to give way in the symbol name space here ...
I remember PIC being dodgy on 32 bit x86 before (register starvation during build), but with current gcc, it works. Anyhow, I think it is a good idea to avoid this conflict and propose the attached patch. Of course, I'm fine with any re-name that doesn't conflict, I just chuckled a bit and turned PIC into EPIC.
(I selected group 0.72, as that suffered from the same issue, as I recall, but I'm on 0.74.)