[PATCH] D9398: llvm.noalias - GetUnderlyingObjects to optionally collect noalias calls

[David Majnemer via llvm-commits]

majnemer added inline comments.


> ValueTracking.cpp:3070-3074
> +      if (auto *I = dyn_cast<IntrinsicInst>(V))
> +        if (I->getIntrinsicID() == Intrinsic::noalias) {
> +          if (NoAlias)
> +            NoAlias->push_back(I);
> +        }

I think you can sink this into the block for the `auto CS = CallSite(V)` condition.  You can compare `CS.getIntrinsicID()` with `Intrinsic::noalias`.

https://reviews.llvm.org/D9398