[cfe-commits] r129465 - in /cfe/trunk: lib/Sema/SemaExpr.cpp test/FixIt/no-macro-fixit.c

jahanian fjahanian at apple.com
Wed Apr 13 15:24:18 PDT 2011


On Apr 13, 2011, at 2:25 PM, jahanian wrote:

> 
> On Apr 13, 2011, at 2:18 PM, Douglas Gregor wrote:
> 
>> 
>> On Apr 13, 2011, at 2:02 PM, Eli Friedman wrote:
>> 
>>> On Wed, Apr 13, 2011 at 1:31 PM, Fariborz Jahanian <fjahanian at apple.com> wrote:
>> 
>> Actually, this is strange: I thought this code already existed. I wonder why the Fix-Its with macro-instantiation source locations aren't getting implicitly dropped appropriately?
> 
> Because, it turns out to eventually become a null SourceLocation. I think the right fix is to pass the macro-instantiation source locations  and see if it magically
> drops the fixit. I will take a look.

In r129481.

- fariborz


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110413/f9669028/attachment.html>


More information about the cfe-commits mailing list