[PATCH] [RFC] Lame fix for PR20376
Peter Collingbourne
peter at pcc.me.uk
Tue Jul 22 11:18:49 PDT 2014
On Tue, Jul 22, 2014 at 10:26:29AM +0000, Tim Northover wrote:
> We actually do have a last-ditch EFLAGS copy available (though with worrying implications for the red-zone, if we ever start using it). It's used by test/CodeGen/Generic/2011-07-07-ScheduleDAGCrash.ll for example.
>
> Enabling it more generally is probably just a matter of tweaking getCopyCost. Which might be an idea for robustness anyway.
I see. That might be reasonable if we don't expect this situation to come
up much.
> But I'm not sure that actually affects the best solution here. I rather suspect performing the comparison again would be quicker in general. I suppose the question is how common and performance-critical such fragments are.
I don't understand this point. Compare exchanges are not idempotent.
Thanks,
--
Peter
More information about the llvm-commits
mailing list