[PATCH] D29203: Change debug-info-for-profiling from a TargetOption to a function attribute.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 17:12:03 PST 2017


Dehao - if you could move it back, that'd be great. no need to send a
review, you can commit it directly

On Wed, Feb 1, 2017 at 5:09 PM David Blaikie <dblaikie at gmail.com> wrote:

> Yeah - test inputs need to be in an Inputs directory. Sorry I didn't
> notice the comment/effect.
>
> On Wed, Feb 1, 2017 at 5:08 PM Dehao Chen via Phabricator <
> reviews at reviews.llvm.org> wrote:
>
> danielcdh added a comment.
>
> This change was addressing one of the review comments.
>
> test/DebugInfo/X86/gmlt.test do now depends on the input file from
> test/DebugInfo/Generic/gmlt.ll. If this is an issue, I can send another
> patch to move gmlt.ll back to test/DebugInfo/Input and add
> test/DebugInfo/Generic/gmlt.test back.
>
>
> https://reviews.llvm.org/D29203
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170202/cb404a30/attachment.html>


More information about the llvm-commits mailing list