Re: [Tinycc-devel] Add missing scope modifier?
grischka wrote Better leave alone tcccoff.c too. Other than that I think it makes sense. You could test with $ make clean make all cross ONE_SOURCE=yes and with "no" too. I followed these instruction, and dumped symbol table of all object files. I found more function has scope problem, and also some function are already have scope modifier in declaration, then I update this patch. the new patch show below. https://github.com/partic2/tinycc/commit/d68ac0b56e422dfef0f18933947bd4151ed232a6___ Tinycc-devel mailing list Tinycc-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/tinycc-devel
Re: [Tinycc-devel] Add missing scope modifier?
On 11.08.2022 17:06, pursuer2 via Tinycc-devel wrote: I'm sorry to send a duplicate mail by mistack a short while ago. what I really want to say is that I make a patch to add all missing scope modifier . Static scope is userful when use ONE_SOURCE build mode, which can avoid unexpected name confilict. The patch show below: https://github.com/partic2/tinycc/commit/2cb29cf0eb29f4e8d79b3f1774c6b3df749af550 But the il and c67 backend are not change because I'm not sure how to test. Better leave alone tcccoff.c too. Other than that I think it makes sense. You could test with $ make clean && make all cross ONE_SOURCE=yes and with "no" too. -- gr And will mob accept this patch? ___ Tinycc-devel mailing list Tinycc-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/tinycc-devel ___ Tinycc-devel mailing list Tinycc-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/tinycc-devel
