[cfe-commits] Use LLVM's existing IR support for fast-math flags during (IR) Codegen

Eric Christopher echristo at gmail.com
Thu Nov 29 14:51:23 PST 2012


Yep. That's what I meant. :)

Noticed the testcases changed between revisions?

-eric


On Thu, Nov 29, 2012 at 2:45 PM, Michael Ilseman <milseman at apple.com> wrote:

>
> On Nov 29, 2012, at 2:29 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
>
> > On Thu, Nov 29, 2012 at 2:18 PM, Michael Ilseman <milseman at apple.com>
> wrote:
> >>
> >> On Nov 29, 2012, at 2:17 PM, Eric Christopher <echristo at gmail.com>
> wrote:
> >>
> >> On Thu, Nov 29, 2012 at 2:12 PM, Michael Ilseman <milseman at apple.com>
> wrote:
> >>>
> >>> Sorry, fixed patch attached
> >>>
> >>
> >> Testcases maybe in the patch next time? :)
> >>
> >>
> >> There were two new test cases attached to the original email.
> >
> > We generally prefer the patch to exactly match what you intend to
> > commit; this means you should include the testcases within the patch
> > file.
> >
>
> Oh, I understand what Eric meant now. Sorry about that! Here's an
> all-included patch.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121129/10ff33a7/attachment.html>


More information about the cfe-commits mailing list