On Sep 4, 2015, at 2:45 PM, Ilya Verbin <[email protected]> wrote: > 2015-09-04 22:27 GMT+03:00 Mike Stump <[email protected]>: >> On Sep 4, 2015, at 4:10 AM, Hahnfeld, Jonas <[email protected]> >> wrote: >>>>>> * intelmic-mkoffload.c (prepare_target_image): Fix if the temp path >>>>>> contains a '-‘. >> >> So, out of curiosity, did you test all characters other than null? If - >> doesn’t work, there is a good chance that no such test has been done, and >> there is a small hoard of bugs, all the same in there. > > Good point. Objcopy in bfd/binary.c creates symbol names this way: > > for (p = buf; *p; p++) > if (! ISALNUM (*p)) > *p = '_’;
So, this code can’t be the code in question, as - is ! ALNUM, and then should have been a _, meaning ‘-‘ would not have to be handled. Since the code is handling ‘-‘, this can’t be the code in question?
