[LLVMdev] dmb ishld in AArch64

Song, Chengyu csong84 at gatech.edu
Wed Dec 10 10:15:11 PST 2014


Yes. I observed another incorrect reordering from the revision I use, which is also fixed in the trunk version.

Thank you so much for the help!

Best,
Chengyu

> On Dec 10, 2014, at 13:06, Tim Northover <t.p.northover at gmail.com> wrote:
> 
> On 9 December 2014 at 16:29, Tim Northover <t.p.northover at gmail.com> wrote:
>>> Indeed. I didn't pay too much attention to the str -> stp change.
>> 
>> Oh good. I can reproduce it now, on the build you were using. Still
>> only with Clang rather than llc, but that's just a minor
>> inconvenience. It should still be solvable.
> 
> I think this was an intentional fix, rather than just some random
> change concealing the issue (the improvement bisected down to Tom's
> r223717, which fits both by description and location). So I think we
> can probably stop worrying about this one.
> 
> Cheers.
> 
> Tim.




More information about the llvm-dev mailing list