[PATCH] D26716: DAG: Ignore call site attributes when emitting target intrinsic

[Eli Friedman via llvm-commits]

efriedma added a comment.

llvm.sqrt is a readnone intrinsic which can exhibit undefined behavior, but I think we don't actually generate a chain for it at the moment, so maybe we can get away with querying whether the intrinsic is readnone.  But the right way to do that is to call Intrinsic::getAttributes; we can't trust the IR to have the right attributes on the declaration (it's not something the verifier checks).


https://reviews.llvm.org/D26716