[llvm-commits] [llvm-gcc-4.2] r64584 - /llvm-gcc-4.2/trunk/gcc/cgraphunit.c

Chris Lattner clattner at apple.com
Sun Feb 15 10:09:00 PST 2009


On Feb 15, 2009, at 6:15 AM, Anton Korobeynikov wrote:

> Hi, Bill
>
>> Fix build failure. Can't embed a directive within a macro argument.
> This looks rather hacky and definitely something not suitable for non-
> llvm build. Why don't just move directive before gcc_assert() and have
> two differen gcc_assert() calls?

Yes, I agree with Anton.  Please don't change the behavior of the non- 
llvm build.

-Chris



More information about the llvm-commits mailing list