[PATCH] Arm: Don't define a label twice with two setjmps in a function.

Renato Golin renato.golin at linaro.org
Wed Jun 24 05:46:17 PDT 2015


Hi Matthias,

Sorry, this fell out of my radar. Now it's a lot clearer what do you mean by "the comment" and why are you using the directional labels. It looks ok to me, but I'm adding some EH and Darwin folks, as this affects them more than anything.

I just have one question: can you create your temporary labels with some prefix? Like "SJLJEH".1 .2 .3 etc? That'd help when reading the resulting assembly.

cheers,
--renato


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9314

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list