[LLVMdev] Botched Build
David Greene
dag at cray.com
Mon Feb 15 13:17:43 PST 2010
On Monday 15 February 2010 15:08:22 Chris Lattner wrote:
> On Feb 15, 2010, at 1:04 PM, David Greene wrote:
> >> FWIW, this is because you broke the encoding of an instruction in your
> >> patch. This is incorrect:
> >>
> >> +def MOVNTDQ_64mr : PSI<0xE7, MRMDestMem, (outs), (ins f128mem:$dst,
> >> VR128:$src), + "movntdq\t{$src, $dst|$dst, $src}",
> >> + [(alignednontemporalstore (v2f64 VR128:$src),
> >> addr:$dst)]>;
> >>
> >> Please don't check in patches when you know that they break testcases.
> >
> > I certainly didn't know it broke anything. I even checked the buildbot
> > and it looked like it failed before I ever applied anything. Apologies
> > for the mix- up.
>
> FYI, if you want to check the encoding of an instruction, you can do so
> with a command like this:
Thanks. It's strange that the buidbot shows this test failing long before I
made the change. Why would that be?
In any event, I've got the problem fixed. It was a cut-n-paste oversight on
my part.
As for documentation, where should that go in LangRef.html? There's
nothing there yet that really talks about special metadata. I can make up
a section if that seems appropriate.
-Dave
More information about the llvm-dev
mailing list