[PATCH] D112942: target ABI: improve call parameters extensions handling

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

"signext" doesn't really make sense on llvm.memset: it's not a call.  It's an intrinsic.  It might be lowered to a call, but that's not really relevant: even if we do generate a call, there's no reason to expect that call's calling convention is the same as memset's calling convention.  instcombine transferring it is just an accident.

The ASan instrumentation pass should be able to figure out the correct attributes for any functions it calls from first principles (possibly using TargetLibraryInfo::getExtAttrForI32Param(), like other instrumentation passes).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112942/new/

https://reviews.llvm.org/D112942