[LLVMdev] RFC: Bugpoint Patch

[Bill Wendling]

On Thu, Feb 26, 2009 at 9:36 AM, Dale Johannesen <dalej at apple.com> wrote:
>
> On Feb 26, 2009, at 1:40 AMPST, Bill Wendling wrote:
>
>> On Feb 25, 2009, at 6:01 PM, Dale Johannesen wrote:
>>
>>> On Feb 25, 2009, at 4:59 PMPST, Bill Wendling wrote:
>>>
>>>> I'm running into a problem where I need to have the "gcc" that's
>>>> executed by "bugpoint" take certain arguments that aren't applicable
>>>> to "llc". So, I came up with this patch, that adds a new flag
>>>> "-gcc-tool-args". It's similar to "-tool-args" but passes the
>>>> command
>>>> line arguments to gcc instead of to llc.
>>>>
>>>> Comments?
>>>
>>> I like the new flag but I'm not sure changing that test is the right
>>> way to fix this failure.  There is nothing inherent in that test
>>> that requires it to be compiled 32-bit; it ought to work in
>>> environments where llvm and the host compiler both target 64-bit
>>> (and I think it does).
>>
>> I agree.
>>
>>> I think if you're going to change the test you need to add "target"
>>> lines to the test to force it to be compiled 32-bit.
>>
>> Unfortunately, we don't have that mechanism available for C or C++
>> files.
>
> ? the failing test is BugPoint/misopt-basictest.ll
>
I was unclear. Shouldn't type at 1AM. :) Anyway, yes, it's a .ll file
problem. I might be able to coerce bugpoint to do the right thing.

-bw