[llvm] r178170 - Fix typo (common to both X86 and PPC)

Hal Finkel hfinkel at anl.gov
Wed Mar 27 13:11:09 PDT 2013


----- Original Message -----
> From: "Michael Liao" <michael.liao at intel.com>
> To: "Hal Finkel" <hfinkel at anl.gov>
> Cc: llvm-commits at cs.uiuc.edu
> Sent: Wednesday, March 27, 2013 2:57:32 PM
> Subject: Re: [llvm] r178170 - Fix typo (common to both X86 and PPC)
> 
> thanks for correcting that, :)

No problem.

By the way, does the X86 SjLj code work correctly with functions with dynamic stack realignment. If you need a base pointer, is that saved/restored? (just from looking through the code, that was not clear to me because it would be reserved, and I don't know if those are always spilled).

 -Hal

> 
> - michael
> 
> On Wed, 2013-03-27 at 19:10 +0000, Hal Finkel wrote:
> > Author: hfinkel
> > Date: Wed Mar 27 14:10:42 2013
> > New Revision: 178170
> > 
> > URL: http://llvm.org/viewvc/llvm-project?rev=178170&view=rev
> > Log:
> > Fix typo (common to both X86 and PPC)
> > 
> > Thanks to Bill Schmidt for pointing this out during code review!
> > 
> > Modified:
> >     llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp
> >     llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
> > 
> > Modified: llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp
> > URL:
> > http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp?rev=178170&r1=178169&r2=178170&view=diff
> > ==============================================================================
> > --- llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp (original)
> > +++ llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp Wed Mar 27
> > 14:10:42 2013
> > @@ -212,7 +212,7 @@ PPCTargetLowering::PPCTargetLowering(PPC
> >    setOperationAction(ISD::EXCEPTIONADDR, MVT::i32, Expand);
> >    setOperationAction(ISD::EHSELECTION,   MVT::i32, Expand);
> >  
> > -  // NOTE: EH_SJLJ_SETJMP/_LONGJMP supported here is NOT intened
> > to support
> > +  // NOTE: EH_SJLJ_SETJMP/_LONGJMP supported here is NOT intended
> > to support
> >    // SjLj exception handling but a light-weight setjmp/longjmp
> >    replacement to
> >    // support continuation, user-level threading, and etc.. As a
> >    result, no
> >    // other SjLj exception interfaces are implemented and please
> >    don't build
> > 
> > Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
> > URL:
> > http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=178170&r1=178169&r2=178170&view=diff
> > ==============================================================================
> > --- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
> > +++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Wed Mar 27
> > 14:10:42 2013
> > @@ -470,7 +470,7 @@ X86TargetLowering::X86TargetLowering(X86
> >      setOperationAction(ISD::SETCC         , MVT::i64  , Custom);
> >    }
> >    setOperationAction(ISD::EH_RETURN       , MVT::Other, Custom);
> > -  // NOTE: EH_SJLJ_SETJMP/_LONGJMP supported here is NOT intened
> > to support
> > +  // NOTE: EH_SJLJ_SETJMP/_LONGJMP supported here is NOT intended
> > to support
> >    // SjLj exception handling but a light-weight setjmp/longjmp
> >    replacement to
> >    // support continuation, user-level threading, and etc.. As a
> >    result, no
> >    // other SjLj exception interfaces are implemented and please
> >    don't build
> > 
> > 
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> 
> 



More information about the llvm-commits mailing list