[PATCH] ValueMapper: Eliminate cross-file co-recursion, NFC

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 07:33:28 PDT 2016


> you can probably use std::find for getMappingContextID. I wonder if it
> wouldn't be cleaner to have the caller do something like
>
> unsigned NormalID = Mapper.registerContext(...);
> unsigned AliasID = Mapped.registerContext(...);
>
> and pass the context to scheduleMap*.

Or just pass an ArrayRef of MappingContext to the constructor?

Cheers,
Rafael


More information about the llvm-commits mailing list