[patch][PR10367] Fix the design of GlobalAlias

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed May 14 09:29:05 PDT 2014


>> Yes. I started going that way, but the churn is so big that it is
>> probably better if we do that as a separate patch. I will send one
>> with just the renaming once these are done.
>
> Would that constitute a C API change?  I'm thinking specifically about
> include/llvm-c/Core.h:

Yes. To be strict about the backwards compatibility we would need to have

LLVMIsAGlobalValue == LLVMIsAGlobalName
LLVMIsAGlobalValueV2 == checks for llvm::GlobalValue

which is pretty nasty. Not sure if we want that or if we can change
the meaning a bit.

In any case, it should be a nice topic for a followup thread :-)

Cheers,
Rafael



More information about the llvm-commits mailing list